Fix i18n files keys sort #14433

sampaiodiego · 2019-05-08T20:36:14Z

Added an script to do that as well.

@mrsimpson what do you think?

mrsimpson

The result looks fine, I also like the idea of using a source language as "sorting template". However, I consider removing non-existent keys at least as "dangerous".

Would you add a script in the package.json (translation-fix) as well?

mrsimpson · 2019-05-08T21:10:22Z

.scripts/fix-i18n.js

+ *
+ * - remove any duplicated i18n key on the same file;
+ * - re-order all keys based on source i18n file (en.i18n.json)
+ * - remove all keys not present in source i18n file


I have seen cases where a "translation" was not maintained in English for single words, where the key is fine as fallback (like _ "ok").
In those cases, I had to add a translation which did not exist in EN. => Until there's a verification that all keys contain a translation in EN, I'd not remove the translations

replying to myself: if someone figures out a translation was missing in EN (but the key was used as display text), he/she/it should add the translation to the EN file anyway. I've done the same earlier on as well and now it seems as if there were no translations missing in EN. => Imho, this is fine.

yes.. also, if some translation is missing from EN nobody will be able to translate it from lingohub, and on the next export from lingohub it would be removed from any other language as well.. so this just implements lingohub's behavior

mrsimpson · 2019-05-08T21:11:31Z

.scripts/fix-i18n.js

+ * - remove all keys not present in source i18n file
+ *
+ * you might need to do:
+ * npm i --no-save fast-glob


Why not add it as a dev dependency?

mrsimpson · 2019-05-08T21:17:45Z

.scripts/fix-i18n.js

+
+			const json = JSON.parse(fs.readFileSync(file, 'utf8'));
+
+			fs.writeFileSync(file, `${ JSON.stringify(json, sourceKeys, 2) }${ newlineAtEnd ? '\n' : '' }`);


I didn't know the feature of the replacer as whitelist in stringify. A comment might help, but it may be an individual incompetence of myself in this area ;)

mrsimpson

Re-thinking my concerns I came to the conclusion that also deleting keys is fine.
Btw: There's already a script translation-diff which compares the keys independent of order.

sampaiodiego · 2019-05-09T11:50:04Z

thanks @mrsimpson .. I was not willing to add it as an "official script", that's why I haven't added a script entry nor added to dev dependencies.. but now I guess I should.. 😛 I'll update the PR doing it.. thx

* Add script to normalize i18n files * Fix i18n files * Set as official script * Update package-lock.json

* [FIX] New day separator overlapping above system message (#14362) * Improve German translations (#14351) * Use the plural for discussions-section in side panel * Formal and informal translations for 1.0 * fix german typos * [FIX] Main thread title on replies (#14372) * fix * fix test * fix setting * Update tests/pageobjects/main-content.page.js Co-Authored-By: ggazzo <guilhermegazzo@gmail.com> * Update app/ui-utils/client/lib/RoomHistoryManager.js Co-Authored-By: ggazzo <guilhermegazzo@gmail.com> * [FIX] Bell was too small on threads (#14394) * [FIX] Messages on threads disappearing (#14393) * fix subscription-changed updating all messages(#14391) * Fix: Message body was not being updated when user disabled nrr message (#14390) * [NEW] Allow change Discussion's properties (#14389) * [FIX] Unnecessary meteor.defer on openRoom (#14396) * [FIX] more message actions to threads context(follow, unfollow, copy, delete) (#14387) * added more message actions to threads context * more actions * change token name (#14379) * [FIX] Pressing Enter in User Search field at channel causes reload (#14388) * Prevent default on enter in User search * Prevent form submission in membersList * If using subpath make sure streams use that also for multi-instance. Fixes #13200 (#14376) * Revert "[IMPROVE] Use SessionId for credential token in SAML request (#13791)" (#14345) This reverts commit 3967a74. * Add fallback to mongo version that doesn't require clusterMonitor role (#14403) * [FIX] Users actions in administration were returning error (#14400) * Fix actions collapse into popup in userInfo * Refactor userActions * [FIX] Error 400 on send a reply to an old thread (#14402) * fix error 400 on send a reply to an old thread * ignoring properly hidden messages * [FIX] Messages on thread panel were receiving wrong context/subscription (#14404) * [FIX] preview pdf its not working (#14419) * [FIX] renderMessageBody was caching messages in wrong scenarios #14420 * LingoHub Update 🚀 (#14426) Manual push by LingoHub User: Diego Sampaio. Project: Rocket.Chat Made with ❤️ by https://lingohub.com * [FIX] Mentions message missing 'jump to message' action (#14430) * fixed context * threads context * [FIX] Escape unrecognized slash command message (#14432) * Add missing german translations (#14386) * [FIX] IE11 support (#14422) * Add symlinks to ES6 node_modules imports * Add URL polyfill for IE11 * Fix thread replies for IE11 * [IMPROVE] allow users to skip activeUsers to be ready (#14431) * allow users to skip activeUsers to be ready * Update main.js * Update app/ui-master/client/main.js Co-Authored-By: ggazzo <guilhermegazzo@gmail.com> * [IMPROVE] Don't use regex to find users (#14397) * Don't use regex to find users * Invert logic on model methods * Escape username regex * Find users in batch * Use only normalizeMessagesForUser * Don't ignore username case to get owners on graphql * Fixes on DAU and MAU aggregations (#14418) * Fixes on SAU and MAU aggregations * Report new data from DAU/MAU * Run tests agains a mongodb container in CI * Try to run CI correctly * Fix drop database * Parse desktop app User Agent correctly * Fix aggregation of past sessions * Return past month today * Fix bug * Add migration * Fixed migration * Migration improvements * Fix crowd sync by using correct logging method (#14405) * Fix room names in user info dialogs (#14415) * Fix discussion name being invalid (#14442) Closes #14378 * Fix i18n files keys sort (#14433) * Add script to normalize i18n files * Fix i18n files * Set as official script * Update package-lock.json * fix (#14443) * Update threads.css * Bump version to 1.0.3 * regen changelog

sampaiodiego added 2 commits May 8, 2019 14:34

Add script to normalize i18n files

8f0492c

Fix i18n files

c723b03

sampaiodiego added this to the 1.0.3 milestone May 8, 2019

sampaiodiego requested review from mrsimpson and rodrigok May 8, 2019 20:36

mrsimpson reviewed May 8, 2019

View reviewed changes

mrsimpson self-requested a review May 8, 2019 21:35

mrsimpson previously approved these changes May 8, 2019

View reviewed changes

Set as official script

266e58d

sampaiodiego dismissed mrsimpson’s stale review via 266e58d May 9, 2019 12:03

sampaiodiego requested a review from ggazzo May 9, 2019 15:32

Merge branch 'develop' into fix-i18n-files

8e33d57

ggazzo previously approved these changes May 9, 2019

View reviewed changes

Update package-lock.json

ba9a495

sampaiodiego dismissed ggazzo’s stale review via ba9a495 May 9, 2019 18:50

ggazzo approved these changes May 9, 2019

View reviewed changes

rodrigok approved these changes May 9, 2019

View reviewed changes

sampaiodiego merged commit f17763f into develop May 9, 2019

sampaiodiego deleted the fix-i18n-files branch May 9, 2019 20:23

rodrigok pushed a commit that referenced this pull request May 9, 2019

Fix i18n files keys sort (#14433)

484d900

* Add script to normalize i18n files * Fix i18n files * Set as official script * Update package-lock.json

rodrigok mentioned this pull request May 9, 2019

Release 1.0.3 #14446

Merged

sampaiodiego mentioned this pull request May 28, 2019

Release 1.1.0 #14653

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix i18n files keys sort #14433

Fix i18n files keys sort #14433

sampaiodiego commented May 8, 2019

mrsimpson left a comment

mrsimpson May 8, 2019

mrsimpson May 8, 2019

sampaiodiego May 9, 2019 •

edited

mrsimpson May 8, 2019

mrsimpson May 8, 2019

mrsimpson left a comment

sampaiodiego commented May 9, 2019


		const json = JSON.parse(fs.readFileSync(file, 'utf8'));

		fs.writeFileSync(file, `${ JSON.stringify(json, sourceKeys, 2) }${ newlineAtEnd ? '\n' : '' }`);

Fix i18n files keys sort #14433

Fix i18n files keys sort #14433

Conversation

sampaiodiego commented May 8, 2019

mrsimpson left a comment

Choose a reason for hiding this comment

mrsimpson May 8, 2019

Choose a reason for hiding this comment

mrsimpson May 8, 2019

Choose a reason for hiding this comment

sampaiodiego May 9, 2019 • edited

Choose a reason for hiding this comment

mrsimpson May 8, 2019

Choose a reason for hiding this comment

mrsimpson May 8, 2019

Choose a reason for hiding this comment

mrsimpson left a comment

Choose a reason for hiding this comment

sampaiodiego commented May 9, 2019

sampaiodiego May 9, 2019 •

edited