Skip to content
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

Update translations: #3390 #3405

Merged
merged 1 commit into from
Apr 11, 2024
Merged

Update translations: #3390 #3405

merged 1 commit into from
Apr 11, 2024

Conversation

translation-platform[bot]
Copy link
Contributor

This PR adds async translations requested in #3390, which was opened by @QuintonC.

It will append multiple commits and may add, modify, or delete translations. Feel free to merge the PR any time; we'll open a new PR if any more translations come in.

Check the source PR's overall translation progress here.

Before you deploy

  • Check translations for malicious HTML.

Merge conflicts?

If this PR has been open long enough that it has drifted out of sync from the base branch and it has developed merge conflicts as a result, follow these steps, in order:

  1. Close this PR.
  2. Run the following Spy command in Slack:

@spy translation-platform force-pull Shopify/dawn pull_request_number=3390

Translation Platform will recreate the translation commits based on the latest commit on the base branch and open a new PR.

Missing completed translations?

If the translations were completed within the last several hours, Translation Platform may just be holding on to them for a bit to see if any others come in before creating a commit (multiple file changes are committed together to reduce noise).

If it's been a while and completed translations are still missing, or if you're just in a hurry, you can run the following Spy command in Slack to download or re-download any completed translations to this PR:

@spy translation-platform force-pull Shopify/dawn pull_request_number=3390

  • The translations might have been added to the source PR, but the PR was rebased before it was shipped and the translations were accidentally dropped.
  • A later change to the translation files might have overwritten the previously downloaded translations.
  • Translation Platform might have failed to download the translations-- if you suspect this to be the case, please reach out at the link below.

Questions?

Visit #help-localization for questions about this pull request.

Please tag the translation component owners, @Shopify/themes-translations, if you need someone to review this PR.

@translation-platform
Copy link
Contributor Author

🕐 This PR has been open for 2 days now 🕐
@QuintonC

It is best to merge it as soon as possible in order to avoid having to deal with merge conflicts.

More importantly, the content here really wants to reach production so everyone can benefit from it.

It is your responsibility to merge this PR once you've reviewed it. 🙏

💚

@sofiamatulis sofiamatulis merged commit ce39a3c into main Apr 11, 2024
6 of 8 checks passed
@sofiamatulis sofiamatulis deleted the translation-platform/pr/3390 branch April 11, 2024 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants