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.

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

Update strings in Android App #186

Merged
merged 7 commits into from
Jan 10, 2024
Merged

Update strings in Android App #186

merged 7 commits into from
Jan 10, 2024

Conversation

rtibbles
Copy link
Member

@rtibbles rtibbles commented Jan 10, 2024

  • Removes unneeded resource strings
  • Fixes residual private version code from p4a that would have prevented the app from upgrading
  • Finds translated user text for every user facing text apart from the Persistent Foreground notification channel, for which I could not find an appropriate string in Kolibri
  • Updates the create_string script to search through Kolibri message files and po files to find translations for messages
  • Updates the generated username to "Learner" translated
  • Caches the above generated username to avoid weird behaviour if the user switches their Android device language

Fixes #172
Fixes #185

Screenshot_20240109_170543

@rtibbles rtibbles requested review from bjester and jredrejo January 10, 2024 01:09
Copy link
Member

@bjester bjester left a comment

Choose a reason for hiding this comment

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

LGTM

@rtibbles rtibbles merged commit 3340ce1 into develop Jan 10, 2024
4 checks passed
@rtibbles rtibbles deleted the cheeky_strings branch January 10, 2024 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Untranslated UI text Randomized usernames are confusing
2 participants