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
nautilus: mgr/dashboard: Add transifex-i18ntool #31160
Conversation
Adds the npm package transifex-i18ntool which manages the translation files of the ceph dashboard Backporting the transifex-i18ntool is an important step to synchronize the nautilus translations with transifex more or less automatically. The npm scripts still need to be started manual, but merging the xlf files etc. will be handled by the i18ntool. Signed-off-by: Sebastian Krah <skrah@suse.com> (cherry picked from commit d33b5d3) Conflicts: src/pybind/mgr/dashboard/frontend/package-lock.json - Had to be updated since the devDependencies in the package.json changed src/pybind/mgr/dashboard/frontend/package.json - Added the transifex-i18ntool as dependency and some scripts to automate the i18n workflow src/pybind/mgr/dashboard/run-frontend-unittests.sh - The way how to extract source language file changed the and had to be adapted New file: src/pybind/mgr/dashboard/frontend/i18n.config.json - Stores the settings for the transifex-i18ntool
Since the settings will divide from master the resource must be updated. The i18ntool should use the correct resource for the nautilus branch and not the same as master or any other version. Otherwise we would miss translations. Signed-off-by: Sebastian Krah <skrah@suse.com>
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.
LGTM - thank you!
jenkins test dashboard |
6 similar comments
jenkins test dashboard |
jenkins test dashboard |
jenkins test dashboard |
jenkins test dashboard |
jenkins test dashboard |
jenkins test dashboard |
jenkins test dashboard |
1 similar comment
jenkins test dashboard |
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.
lgtm
QA run [1 failed / 3 passed]: New round of QA was successful: |
backport tracker: https://tracker.ceph.com/issues/42482