Skip to content

Conversation

@openfoodfacts-bot
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Nov 20, 2025

❌ 3 Tests Failed:

Tests completed Failed Passed Skipped
7977 3 7974 0
View the top 3 failed test(s) by shortest run time
tests_unit_lang_t::0085 - 'footer_code_of_conduct_link' in 'hr' should be a link
Stack Traces | 0.00271s run time
not ok 0085 - 'footer_code_of_conduct_link' in 'hr' should be a link
tests_unit_i18n_t::1680 - common: 'text_direction' in 'lv': must be 'ltr' or 'rtl'
Stack Traces | 0.00302s run time
not ok 1680 - common: 'text_direction' in 'lv': must be 'ltr' or 'rtl'
tests_integration_external_sources_t::0009 - external-sources-fr
Stack Traces | 0.00477s run time
not ok 0009 - external-sources-fr

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@teolemon teolemon requested a review from Copilot November 23, 2025 20:09
Copilot finished reviewing on behalf of teolemon November 23, 2025 20:12
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR merges new translations from Crowdin for the Open Food Facts project, covering localization updates across 7 files in multiple languages (Russian, Portuguese (Portugal), Polish, Latvian, French, Greek, and Croatian).

  • Adds missing translations for previously untranslated strings
  • Updates donation URLs to use localized paths
  • Corrects minor punctuation inconsistencies

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
po/common/ru.po Adds translations for "Country", "External sources", and "Provided by"
po/common/pt_PT.po Updates donation URL to use fully translated Portuguese path
po/common/pl.po Updates donation URL and adds translation for donated products subtitle
po/common/lv.po Updates text_direction field
po/common/fr.po Removes trailing periods from two strings for consistency
po/common/el.po Updates donation URL to use Greek locale and translated path
html/donate/hr.html Translates page title to Croatian

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

msgctxt "text_direction"
msgid "ltr"
msgstr "ltr"
msgstr "litri"
Copy link

Copilot AI Nov 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The text_direction field should contain the literal value 'ltr' (left-to-right), not the translated word 'litri' (liters in Latvian). This is a technical directive for UI text rendering and must be either 'ltr' or 'rtl', never translated. Change to: msgstr "ltr"

Suggested change
msgstr "litri"
msgstr "ltr"

Copilot uses AI. Check for mistakes.
msgctxt "donate_link"
msgid "https://world.openfoodfacts.org/donate-to-open-food-facts"
msgstr "https://world.openfoodfacts.org/donate-to-open-food-facts"
msgstr "https://world-pl.openfoodfacts.org/pozhertvovat-open-food-facts"
Copy link

Copilot AI Nov 23, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The URL path uses 'pozhertvovat', which is Russian (пожертвовать), not Polish. The donation URL should use a Polish translation. Based on the pattern from other languages (German uses 'spenden', Spanish uses 'dar-a'), this should use an appropriate Polish translation like 'przekaz-darowizne' or similar Polish phrase for 'donate'.

Suggested change
msgstr "https://world-pl.openfoodfacts.org/pozhertvovat-open-food-facts"
msgstr "https://world-pl.openfoodfacts.org/przekaz-darowizne-open-food-facts"

Copilot uses AI. Check for mistakes.
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants