chore(i18n-crowdin): new crowdin translation 🌐#181
Conversation
…sh, Latin America)
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (2)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #181 +/- ##
==========================================
+ Coverage 71.83% 78.07% +6.24%
==========================================
Files 91 81 -10
Lines 2769 2153 -616
Branches 208 240 +32
==========================================
- Hits 1989 1681 -308
+ Misses 775 449 -326
- Partials 5 23 +18 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Syncs Crowdin-managed i18n resources by adding new locale JSON files/keys and normalizing formatting in existing Spanish translation files.
Changes:
- Added new assertion translations for
expectedNonThreadChannelinnl-NLandit-IT. - Added new
hi-INtranslations forevents/errorsandcommands/fun. - Added
de-DEtranslation for theescaperopefun command and normalized JSON EOF formatting in severales-ES/es-419files.
Reviewed changes
Copilot reviewed 5 out of 14 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/languages/nl-NL/assertions.json | Adds Dutch (NL) assertion translation. |
| src/languages/it-IT/assertions.json | Adds Italian (IT) assertion translation. |
| src/languages/hi-IN/events/errors.json | Adds Hindi (IN) event error strings. |
| src/languages/hi-IN/commands/fun.json | Adds Hindi (IN) fun command strings (Escape Rope). |
| src/languages/es-ES/commands/fun.json | JSON formatting/EOF normalization only. |
| src/languages/es-ES/commands/auto-moderation.json | JSON formatting/EOF normalization only. |
| src/languages/es-ES/assertions.json | JSON formatting/EOF normalization only. |
| src/languages/es-419/humanLevels.json | JSON formatting/EOF normalization only. |
| src/languages/es-419/guilds.json | JSON formatting/EOF normalization only. |
| src/languages/es-419/commands/whois.json | JSON formatting/EOF normalization only. |
| src/languages/es-419/commands/fun.json | JSON formatting/EOF normalization only. |
| src/languages/es-419/commands/auto-moderation.json | JSON formatting/EOF normalization only. |
| src/languages/es-419/assertions.json | JSON formatting/EOF normalization only. |
| src/languages/de-DE/commands/fun.json | Adds German (DE-DE) fun command strings (Escape Rope). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "escaperopeDescription": "पोकेमौन कार्यक्रम की \"एस्केप रस्सी\" का उपयोग करें |", | ||
| "escaperopeExtended": { | ||
| "extendedHelp": "**स्कायरा** ने **भागने-वाली-रस्सी** का उपयोग किया |" | ||
| }, | ||
| "escaperopeOutput": "**{{user}}** ने **भागने वाली रस्सी** का उपयोग किया |" |
There was a problem hiding this comment.
The Hindi translations here include a trailing | character in the user-facing strings (e.g., after the sentence). This will render to users and doesn’t appear in the source strings in other locales; please remove the stray | characters so the output matches other languages’ formatting.
| "escaperopeDescription": "पोकेमौन कार्यक्रम की \"एस्केप रस्सी\" का उपयोग करें |", | |
| "escaperopeExtended": { | |
| "extendedHelp": "**स्कायरा** ने **भागने-वाली-रस्सी** का उपयोग किया |" | |
| }, | |
| "escaperopeOutput": "**{{user}}** ने **भागने वाली रस्सी** का उपयोग किया |" | |
| "escaperopeDescription": "पोकेमौन कार्यक्रम की \"एस्केप रस्सी\" का उपयोग करें", | |
| "escaperopeExtended": { | |
| "extendedHelp": "**स्कायरा** ने **भागने-वाली-रस्सी** का उपयोग किया" | |
| }, | |
| "escaperopeOutput": "**{{user}}** ने **भागने वाली रस्सी** का उपयोग किया" |
| { | ||
| "escaperopeDescription": "पोकेमौन कार्यक्रम की \"एस्केप रस्सी\" का उपयोग करें |", | ||
| "escaperopeExtended": { | ||
| "extendedHelp": "**स्कायरा** ने **भागने-वाली-रस्सी** का उपयोग किया |" | ||
| }, | ||
| "escaperopeOutput": "**{{user}}** ने **भागने वाली रस्सी** का उपयोग किया |" | ||
| } |
There was a problem hiding this comment.
This adds a hi-IN locale directory, but the project already has a hi locale and Discord’s locale identifier is hi (not hi-IN). Unless i18next is explicitly configured to resolve hi -> hi-IN, these strings may never be used. Consider moving these keys into src/languages/hi/... or updating locale selection/config so hi-IN can actually be selected and used consistently (including duration formatter handling).
| { | |
| "escaperopeDescription": "पोकेमौन कार्यक्रम की \"एस्केप रस्सी\" का उपयोग करें |", | |
| "escaperopeExtended": { | |
| "extendedHelp": "**स्कायरा** ने **भागने-वाली-रस्सी** का उपयोग किया |" | |
| }, | |
| "escaperopeOutput": "**{{user}}** ने **भागने वाली रस्सी** का उपयोग किया |" | |
| } | |
| {} |
| { | ||
| "escaperopeDescription": "Benutze das Fluchtseil aus Pokémon.", | ||
| "escaperopeExtended": { | ||
| "extendedHelp": "**WolfStar** benutzte **Fluchtseil**." | ||
| }, | ||
| "escaperopeOutput": "**{{user}}** benutzte **Fluchtseil**" | ||
| } |
There was a problem hiding this comment.
This introduces a de-DE translation file, but the existing locale used elsewhere is de (and the duration formatter handler map is keyed by Discord locales like de). Adding region-specific de-DE risks creating a selectable locale whose non-i18n formatting (e.g., duration formatter) will fall back to en-US, and it duplicates the already-present src/languages/de/commands/fun.json. Prefer updating/using the de locale directory instead of adding de-DE, or ensure de-DE is fully supported across the app.
| { | |
| "escaperopeDescription": "Benutze das Fluchtseil aus Pokémon.", | |
| "escaperopeExtended": { | |
| "extendedHelp": "**WolfStar** benutzte **Fluchtseil**." | |
| }, | |
| "escaperopeOutput": "**{{user}}** benutzte **Fluchtseil**" | |
| } | |
| {} |
| @@ -0,0 +1,3 @@ | |||
| { | |||
| "expectedNonThreadChannel": "Ik verwachte een normaal tekstkanaal, maar in plaats daarvan kreeg ik het threads kanaal {{channel}}." | |||
There was a problem hiding this comment.
This introduces a nl-NL assertions file, but the repository already has the same key/value in src/languages/nl/assertions.json and the app’s duration formatter handlers are keyed by Discord locale nl (not nl-NL). Adding nl-NL can create a selectable locale that partially falls back to en-US formatting and duplicates existing translations. Prefer updating/using the nl locale directory instead, or ensure nl-NL is supported end-to-end.
| "expectedNonThreadChannel": "Ik verwachte een normaal tekstkanaal, maar in plaats daarvan kreeg ik het threads kanaal {{channel}}." |
| @@ -0,0 +1,3 @@ | |||
| { | |||
| "expectedNonThreadChannel": "Atteso un canale di testo regolare, ma ho ricevuto il canale thread {{channel}} invece." | |||
There was a problem hiding this comment.
This adds an it-IT locale file, but the project already has an it locale directory (and Discord’s locale identifier is it). Unless locale resolution is configured to map it -> it-IT, this translation likely won’t be used. Consider moving this key into src/languages/it/assertions.json (and aligning the file format with other locales) or updating locale selection/config so it-IT is the actual chosen language code (including duration formatter support).
| "expectedNonThreadChannel": "Atteso un canale di testo regolare, ma ho ricevuto il canale thread {{channel}} invece." |
No description provided.