Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Plurals aren't translated correctly for Arabic #3829

Closed
BenHenning opened this issue Sep 21, 2021 · 1 comment · Fixed by #3849
Closed

Plurals aren't translated correctly for Arabic #3829

BenHenning opened this issue Sep 21, 2021 · 1 comment · Fixed by #3849
Assignees
@srushtirk
Labels
Z-ibt Temporary label for Ben to keep track of issues he's triaged.
Milestone
0.6 Release Block...

Comments

@BenHenning
Copy link
Sponsor Member

See example plurals in the original strings file: https://github.com/oppia/oppia-android/blob/develop/app/src/main/res/values/strings.xml#L154 & translated in Portuguese: https://github.com/oppia/oppia-android/blob/develop/app/src/main/res/values-pt-rBR/strings.xml#L141 versus Arabic: https://github.com/oppia/oppia-android/blob/develop/app/src/main/res/values-ar/strings.xml#L137. Somehow, the translations in Arabic dropped the plurals.
@BenHenning
Copy link
Sponsor Member Author

/cc @srushtirk & @isalooo & @MohamedMedhat1998

@srushtirk can you look into this & work with our translators to figure out what the discrepancy is & see if we can fix this for Arabic translations by EOD Wed (so that they can be picked up in Thursday's translations drop)?

@BenHenning BenHenning added this to the Alpha MR3 milestone Sep 27, 2021
@BenHenning BenHenning mentioned this issue Sep 27, 2021
Merged
BenHenning added a commit that referenced this issue Sep 28, 2021
@translatewiki @BenHenning
Fix #3829: Localisation updates from https://translatewiki.net. (#3849)
c1d22df 
* Localisation updates from https://translatewiki.net.

* Update strings.xml

Switch digit type specifiers over to strings as now expected.

Co-authored-by: Ben Henning <[email protected]>
@BenHenning BenHenning added the Z-ibt Temporary label for Ben to keep track of issues he's triaged. label Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@srushtirk srushtirk

Labels
Z-ibt Temporary label for Ben to keep track of issues he's triaged.
Projects
[Team] Developer Workflow & Infrastructure - Android
Archived in project
Milestone
0.6 Release Blockers (Alpha MR3)
Development

Successfully merging a pull request may close this issue.

Fix #3829: Localisation updates from https://translatewiki.net. oppia/oppia-android
2 participants
@BenHenning @srushtirk