-
-
Notifications
You must be signed in to change notification settings - Fork 366
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 Locale File Keys #1006
Update Locale File Keys #1006
Conversation
Added missing keys to ar locale files.
add missing keys for de_DE locale files
add missing keys to es locale files
add missing keys to es_ES locale files
add missing keys to fa locale files
add missing keys to fr_FR locale files.
add missing keys to it_IT locale files
add missing keys to pt_PT locale files.
add missing keys to th_TH locale files.
add missing keys to tr locale files
add missing keys for zh_CN locale files.
Find and remove/fix keys present in other locale files but not found in en_US.
Find and remove/fix keys present in other locale files but not found in en_US
Codecov Report
@@ Coverage Diff @@
## develop #1006 +/- ##
==========================================
Coverage 66.99% 66.99%
Complexity 1923 1923
==========================================
Files 161 161
Lines 6703 6703
==========================================
Hits 4491 4491
Misses 2212 2212
Continue to review full report at Codecov.
|
Would it be easier for translators if we comment all untranslated keys? I know it won't work with the upcoming Bakery commands, but it would make it easier to spot what needs translation and actually use the English fallback instead of having outdated English strings in whatever locale. |
I can do that if you think it's best. I do have a table with the untranslated keys that I was going to put into a new issue for translators to reference. |
I commented out the keys in recent commits. |
https://travis-ci.org/userfrosting/UserFrosting/jobs/571120012#L881 |
I'm not even sure where this change came from? |
Something barfed 🤔 |
I don't even understand what happened here... |
I think it should be safe to merge now? PHP-CS-Fixer didn't brought up any issue on my side? |
I think it is good to go. |
Generated missing locale keys for all languages using PR #992 locale:fix-keys command.
The empty key values were populated using en_US and can be identified via locales:missing-values command.
(also wanted to check these files will pass style-ci check)
I also had to go through and modify some files manually to fix keys that were present in other locales but not in en_US. Some of these were due to keys not being in correct sub keys or improper formatting.