-
Notifications
You must be signed in to change notification settings - Fork 2.7k
[Weblate] Remove empty/unchanged translation strings. Add Bulgarian language. Add general translation updates. #6349
Conversation
Updated by "Cleanup translation files" hook in Weblate. Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/
Currently translated at 95.3% (3894 of 4084 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/de/ Added translation using Weblate (Bulgarian) Translated using Weblate (Ukrainian) Currently translated at 77.1% (3148 of 4080 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/uk/ Translated using Weblate (Polish) Currently translated at 90.9% (3711 of 4080 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/pl/ Translated using Weblate (French) Currently translated at 93.5% (3816 of 4080 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/fr/ Translated using Weblate (Ukrainian) Currently translated at 84.2% (3438 of 4080 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/uk/ Translated using Weblate (Ukrainian) Currently translated at 86.3% (3524 of 4080 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/uk/ Translated using Weblate (Ukrainian) Currently translated at 91.9% (3690 of 4011 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/uk/ Translated using Weblate (French) Currently translated at 97.6% (3917 of 4011 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/fr/ Translated using Weblate (German) Currently translated at 97.0% (3894 of 4011 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/de/
Currently translated at 99.8% (4003 of 4011 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/ja/
Currently translated at 0.0% (0 of 4084 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/bg/
Updated by "Cleanup translation files" hook in Weblate. Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/
/update-branch |
i18n/i18n.jsx
Outdated
bg: { | ||
value: 'bg', | ||
name: 'България', | ||
order: 0, |
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.
please update the order of the other languages
This one needs a QA review |
…ermost-webapp into weblate-fix-empty-master
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.
1 - Bulgarian language addition looks good, except please also add test for it at: https://github.com/mattermost/mattermost-webapp/blob/master/components/admin_console/__snapshots__/schema_admin_settings.test.jsx.snap
2 - Question: what does "general translation updates" refer to?
3 - Two other comments below
@@ -824,12 +824,8 @@ | |||
"admin.permissions.permission.manage_oauth.name": "OAuth-2.0-Applikationen verwalten", | |||
"admin.permissions.permission.manage_outgoing_webhooks.description": "Eingehende und ausgehende Webhooks erstellen, bearbeiten und löschen.", | |||
"admin.permissions.permission.manage_outgoing_webhooks.name": "Ausgehende Webhooks verwalten", | |||
"admin.permissions.permission.manage_private_channel_members.description": "Mitglieder zu privaten Kanälen hinzufügen und entfernen.", |
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.
Are these intentionally removed?
"warn_metric_ack_modal.learn_more.link": "Mehr erfahren", | ||
"team.button.name_undefined": "Dieses Team hat keinen Namen", | ||
"team_import_tab.importHelpExportInstructions": "Slack > Administration > Workspace settings > Import/Export Data > Export > Start Export", | ||
"channel_modal.displayNameError": "Anzeigename muss mindestens 2 Zeichen lang sein.", |
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.
These strings are not in alphabetical order.
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.
@metanerd These strings are still not in alphabetical order - will they be resolved in another PR? If so, I'm fine with that.
|
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.
Translations look good although I will say they are quite formal and I thought the language we are keeping is more personal? There are some word shortenings that don't seem necessary IMO. But I won't block the PR just curious :)
Also there's a test failing but I think you only need to update the snapshots.
@jasonblais yes, upcoming PRs will fix the alphabetical order. |
@jgilliam17 this is ready for QA review |
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.
Thank you @metanerd
- Verified Bulgarian language has been added in Account Settings>Display>Language
I noticed few issues with capitalization on account settings modal, extra icons onmore...
links on the old sidebar and the abbreviations and general language improvements Nevy mentioned.
I will create a separate ticket so these translations/changes can be addressed by the contributors.
Can we capitalize the first letter of the language name on the dropdown in this PR?
@@ -340,31 +340,36 @@ exports[`components/admin_console/SchemaAdminSettings should match snapshot with | |||
}, | |||
Object { | |||
"order": 10, | |||
"text": "български (Beta)", |
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.
Can you please capitalize, change български to Ƃългарски.
I didn't use the suggestion here as I didn't want to accidentally introduce a different font.
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.
Thank you @metanerd
- Verified Bulgarian language has been added in Account Settings>Display>Language, language title has been capitalized.
Follow up issue added for translation improvement (capitalization missing, abbreviations etc.) https://mattermost.atlassian.net/browse/MM-28522
Test server destroyed |
…anguage. Add general translation updates. (#6349) * Update translation files Updated by "Cleanup translation files" hook in Weblate. Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/ * Translated using Weblate (German) Currently translated at 95.3% (3894 of 4084 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/de/ Added translation using Weblate (Bulgarian) Translated using Weblate (Ukrainian) Currently translated at 77.1% (3148 of 4080 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/uk/ Translated using Weblate (Polish) Currently translated at 90.9% (3711 of 4080 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/pl/ Translated using Weblate (French) Currently translated at 93.5% (3816 of 4080 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/fr/ Translated using Weblate (Ukrainian) Currently translated at 84.2% (3438 of 4080 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/uk/ Translated using Weblate (Ukrainian) Currently translated at 86.3% (3524 of 4080 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/uk/ Translated using Weblate (Ukrainian) Currently translated at 91.9% (3690 of 4011 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/uk/ Translated using Weblate (French) Currently translated at 97.6% (3917 of 4011 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/fr/ Translated using Weblate (German) Currently translated at 97.0% (3894 of 4011 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/de/ * Translated using Weblate (Japanese) Currently translated at 99.8% (4003 of 4011 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/ja/ * Translated using Weblate (Bulgarian) Currently translated at 0.0% (0 of 4084 strings) Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/bg/ * Update translation files Updated by "Cleanup translation files" hook in Weblate. Translation: mattermost-languages-shipped/mattermost-webapp_master Translate-URL: https://translate.mattermost.com/projects/mattermost/mattermost-webapp_master/ * Remove empty/unchanged translation strings. Add Bulgarian language. * Reorder language order. * Update snapshot. * Change to uppercase. * Update snapshot * Add rudder keys to node env prod. Co-authored-by: Hosted Weblate <[email protected]> Co-authored-by: Elisabeth Kulzer <[email protected]> Co-authored-by: Anonymous <[email protected]> Co-authored-by: Nikolai Zahariev <[email protected]> Co-authored-by: Mattermod <[email protected]>
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.
To soon, did'n check all the files
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.
ok
Summary
Remove empty/unchanged translation strings. Add Bulgarian language. Add general translation updates.Ticket Link
Related Pull Requests
Screenshots