Skip to content

I18n sync Down & Out 04/15 - #51364

Merged
daynew merged 58 commits into
stagingfrom
i18n-sync-down-out-04-15-2023
Apr 17, 2023
Merged

I18n sync Down & Out 04/15#51364
daynew merged 58 commits into
stagingfrom
i18n-sync-down-out-04-15-2023

Conversation

@deploy-code-org

Copy link
Copy Markdown
Contributor

I18n Sync Down & Out

This PR contains all changes to translations made since the last sync (usually a week).

To Review:

Look through the changes in each of the commits and verify that all changes are expected. Note that it is not necessary to review every single "dashboard i18n update" commit. These commits are batched by language to make it easier to review just a couple commits in this category.

Things to Note:

  • We expect translation changes to come in on a regular basis; new strings will be translated, existing translations will be updated, and source string removals will result in translations for those strings also being removed.

  • We expect translations to arrive in large batches, so don't worry if you see a large group of changes in a specific language.

  • We do not usually expect to see a large number of formatting changes, or batches of changes that apply to strings in every single language. If you see this, it usually means either that a code change went in or that something broke.

Known Issues:

  • If the CI unit tests fail at the grunt messages:all step, they will typically fail with a message like:

    [2019-10-04 02:38:57] Running "messages:all" (messages) task
    [2019-10-04 02:39:00] Warning: Error processing localization file i18n/common/zh_tw.json: SyntaxError: Expected [0-9a-zA-Z$_] but "\u8A0A" found. Use --force to continue. 
    

    This usually means that a translator has translated one of the MessageFormat variables. In this case, you should find and fix the offending translation in crowdin, and then manually update the associated strings in i18n/locales and apps/i18n.

    See ab3e5d1 and 0fca376 for an example.

To Deploy:

Once one or two people have reviewed and approved this change and the tests are passing, ship it!

It's not necessary to wait for everyone on the team to review.

@deploy-code-org
deploy-code-org requested a review from a team as a code owner April 15, 2023 01:59
@daynew
daynew merged commit a6b7785 into staging Apr 17, 2023
@daynew
daynew deleted the i18n-sync-down-out-04-15-2023 branch April 17, 2023 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants