-
Notifications
You must be signed in to change notification settings - Fork 5
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
[JN-1416] Add language texts for 10 new languages #1364
Conversation
{"keyName": "navbarDashboard", "text": "Tablero", "language": "es"}, | ||
{"keyName": "navbarChangePassword", "text": "Cambiar Contraseña", "language": "es"}, | ||
{"keyName": "navbarLogout", "text": "Cerrar Sesión", "language": "es"}, | ||
{"keyName": "navbarDashboard", "text": "Panel de control", "language": "es"}, |
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.
This file in particular is interesting to look at because it basically shows us Google Translate versus a professional translation service
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.
nice!
maybe this is totally redundant, but would it be worth making a test that makes sure there are at least some language texts populated for all of these languages? just to make sure something doesn't blow up and we lose a language.
Yeah, that sounds like a reasonable test. I was surprised that we're now escaping |
5e37a66
to
ed03b48
Compare
Quality Gate passedIssues Measures |
DESCRIPTION (include screenshots, and mobile screenshots for participant UX)
Per LanguageScientific, this adds 10 new languages and updates our Spanish language texts.
TO TEST: (simple manual steps for confirming core behavior -- used for pre-release checks)
Restart AdminApiApp
Confirm language texts repopulate without error
select count(*) from language_text;
should yield about 1800 rows