Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

[Weblate] Remove empty/unchanged translation strings. Add Bulgarian language. Add general translation updates. #6349

Merged
merged 21 commits into from
Sep 10, 2020

Conversation

metanerd
Copy link
Contributor

@metanerd metanerd commented Sep 1, 2020

Summary

Remove empty/unchanged translation strings. Add Bulgarian language. Add general translation updates.

Ticket Link

Related Pull Requests

  • Has server changes (please link here)
  • Has redux changes (please link here)
  • Has mobile changes (please link here)

Screenshots

weblate and others added 10 commits August 31, 2020 18:03
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/
@metanerd
Copy link
Contributor Author

metanerd commented Sep 1, 2020

/update-branch

i18n/i18n.jsx Outdated
bg: {
value: 'bg',
name: 'България',
order: 0,
Copy link
Contributor

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

@enahum enahum added the 3: QA Review Requires review by a QA tester label Sep 1, 2020
@enahum
Copy link
Contributor

enahum commented Sep 1, 2020

This one needs a QA review

@metanerd metanerd requested a review from enahum September 1, 2020 12:44
@enahum enahum added 1: PM Review Requires review by a product manager 2: Dev Review Requires review by a core commiter labels Sep 1, 2020
Copy link
Contributor

@jasonblais jasonblais left a 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.",
Copy link
Contributor

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.",
Copy link
Contributor

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.

Copy link
Contributor

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.

@metanerd
Copy link
Contributor Author

metanerd commented Sep 1, 2020

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

  1. Ah nice, will do!
  2. These are the general updates from this PR: Translations update from Weblate #6347 (I just cannot modify that PR. At least, I don't know how)
  3. Thank you, will fix and enable sorting on weblate side.

Copy link
Contributor

@nevyangelova nevyangelova left a 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 jasonblais added this to the v5.28 milestone Sep 7, 2020
@metanerd
Copy link
Contributor Author

metanerd commented Sep 7, 2020

@jasonblais yes, upcoming PRs will fix the alphabetical order.

@jasonblais jasonblais removed 1: PM Review Requires review by a product manager 2: Dev Review Requires review by a core commiter labels Sep 7, 2020
@jasonblais
Copy link
Contributor

@jgilliam17 this is ready for QA review

@jgilliam17 jgilliam17 added the Setup Cloud Test Server Setup a test server using Mattermost Cloud label Sep 8, 2020
Copy link
Contributor

@jgilliam17 jgilliam17 left a 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 on more... 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?

Screen Shot 2020-09-08 at 2 44 27 PM

Screen Shot 2020-09-08 at 3 30 26 PM

@@ -340,31 +340,36 @@ exports[`components/admin_console/SchemaAdminSettings should match snapshot with
},
Object {
"order": 10,
"text": "български (Beta)",
Copy link
Contributor

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.

Copy link
Contributor

@jgilliam17 jgilliam17 left a 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

@jgilliam17 jgilliam17 added 4: Reviews Complete All reviewers have approved the pull request and removed 3: QA Review Requires review by a QA tester Setup Cloud Test Server Setup a test server using Mattermost Cloud labels Sep 10, 2020
@mm-cloud-bot
Copy link

Test server destroyed

@metanerd metanerd merged commit fbb98a7 into master Sep 10, 2020
@metanerd metanerd deleted the weblate-fix-empty-master branch September 10, 2020 08:28
@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Sep 10, 2020
jfrerich pushed a commit that referenced this pull request Oct 23, 2020
…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]>
Copy link
Contributor

@ctlaltdieliet ctlaltdieliet left a 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

Copy link
Contributor

@ctlaltdieliet ctlaltdieliet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
4: Reviews Complete All reviewers have approved the pull request Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.