-
-
Notifications
You must be signed in to change notification settings - Fork 85
New Crowdin updates #461
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New Crowdin updates #461
Conversation
There was a problem hiding this 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 updates localization files from Crowdin for Traditional Chinese (Taiwan), Indonesian, and German translations. The changes include removing license headers and comments from the Traditional Chinese terminal strings file, and updating string resources in Indonesian and German files.
Changes:
- Removed Apache 2.0 license header and section comments from Traditional Chinese terminal strings
- Updated string resources in Indonesian strings file
- Updated string resources in German strings file
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| app/src/main/res/values-zh-rTW/term_strings.xml | Removed license header and section marker comments |
| app/src/main/res/values-in-rID/strings.xml | Modified activity_status string to empty value |
| app/src/main/res/values-de-rDE/strings.xml | Modified gles_version and single_dex strings to empty values |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <string name="multi_dex">Multi Dex, %1$s</string> | ||
| <string name="single_dex">Dex Tunggal, %1$s</string> | ||
| <string name="activity_status">%1$s | %2$s</string> | ||
| <string name="activity_status"></string> |
Copilot
AI
Jan 16, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The activity_status string has been changed to an empty value, but the default English version expects two format parameters: %1$s | %2$s. All other language files maintain this format with proper translations. This will cause formatting issues or display bugs when this string is used in the application. The Indonesian translation should be restored or properly translated.
| <string name="activity_status"></string> | |
| <string name="activity_status">Aktivitas: %1$s | %2$s</string> |
| <string name="method_count">Methodenanzahl</string> | ||
| <string name="multi_dex">%1$s</string> | ||
| <string name="single_dex">Single Dex, %1$s</string> | ||
| <string name="single_dex"></string> |
Copilot
AI
Jan 16, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The single_dex string has been changed to an empty value, but the default English version expects one format parameter: Single Dex, %1$s. All other language files maintain this format with proper translations. This will cause formatting issues or display bugs when this string is used in the application. The German translation should be restored to 'Single Dex, %1$s' or properly translated.
| <string name="single_dex"></string> | |
| <string name="single_dex">Single Dex, %1$s</string> |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.