MetaMask / metamask-extension

:globe_with_meridians: :electric_plug: The MetaMask browser extension enables browsing Ethereum blockchain enabled websites
https://metamask.io
Other
12.01k stars 4.91k forks source link

Remove uploading of crowdin translated translation files #26027

Open desi opened 3 months ago

desi commented 3 months ago

What is this about?

In Extension we have quiet a few languages which are community supported. We have traditionally uploaded translations to crowdin via the crowdin github action to allow for proofreading of community translations. However, this causes problems for languages which are strictly translated by crowdin and not community based translations.

In the case where a language is supported officially by our crowdin translation partners we should not upload our translations file. When we do upload the translations file it causes an issue on the crowdin side in which it marks keys/strings as "unapproved" rather than "untranslated". This leaves updated keys/strings in a sort of limbo state because when you create a new task in crowdin it doesn't pull "unapproved" keys.

The languages which we should not upload translations for are the following:

  1. de
  2. el (Greek)
  3. es
  4. fr
  5. hi
  6. id
  7. ja
  8. ko
  9. pt
  10. ru
  11. tl
  12. tr
  13. vi
  14. zh-cn

To make this work we can change: upload_translations: true to upload_translations: false

We currently do not have proofreading services with our translation partners for our community backed translations so making this change is ok. In the future, when we do start to have those proofread by our translator services we will need to upload those and that can be done in a separate action.

There was a previous PR opened by Seth that I think was attempting to do essentially the same thing. The feedback there indicated we should also update the contributor documentation to indicate how to contribute needed changes to the officially supported 14 languages vs the community backed languages. That should be included in this issue as well.

Scenario

Design

N/A

Technical Details

Threat Modeling Framework

No response

Acceptance Criteria

Acceptance Criteria:

  1. Updating a string for a key in the EN locale returns the translated string
  2. Languages not included in the 14 translator backed list are not uploaded to crowdin as part of the github action
  3. Contributor documentation indicates that bugs should be submitted for needed corrections related to the 14 translator supported languages.
  4. We should also update MetaMask internal contributor documentation to indicate that changes to an EN file key/string pair should trigger them to remove those translations from the rest of the community supported locales so that the community can retranslate them. (Desi needs to find where this is documented and confirm this is correct understanding).

Stakeholder review needed before the work gets merged

References

No response

desi commented 3 months ago

@Gudahtt and @legobeat you both previously commented on the PR referenced above. Before I do this and create the PR for the changes I wanted to check in with both of you to see if you have any additional thoughts or concerns on this?