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

Automate updating gutenberg translations (void) #10807

Merged
merged 57 commits into from
Nov 15, 2019
Merged

Conversation

Tug
Copy link
Contributor

@Tug Tug commented Nov 15, 2019

Fixes wordpress-mobile/gutenberg-mobile#939

Gutenberg PR: wordpress-mobile/gutenberg-mobile#1520

This PR automates adding gutenberg mobile strings to the main strings.xml files in order to have those translated by glotpress. It also adds support for plurals in GutenbergEditorFragment (need to check that glotpress can parse <plurals> tags).

Testing Instructions

  • Checkout the branch locally and make sure the gutenberg-submodule is up to date (or targets develop)
  • Run python tools/merge_strings_xml.py
  • Check the main strings.xml at WordPress/src/main/res/values/strings.xml and make sure it contains all the strings from gutenberg-mobile/bundle/android
  • Build the app to make sure the main xml is valid

PR submission checklist:

  • Update the release process when the PR lands.

  • I have considered adding unit tests where possible.

  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

cameronvoell and others added 30 commits October 29, 2019 18:24
…ute image preview requests from GB editor to MediaPreviewActivity.
Tug and others added 25 commits November 12, 2019 13:48
…creen-preview

Gutenberg image fullscreen preview
…-progressive-rollout

Gutenberg progressive rollout
@Tug Tug self-assigned this Nov 15, 2019
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Nov 15, 2019

Warnings
⚠️ PR is missing at least one label.
⚠️ PR is not assigned to a milestone.

Generated by 🚫 dangerJS

@mchowning mchowning merged commit 9ecc087 into develop Nov 15, 2019
@Tug Tug changed the title Automate updating gutenberg translations (Take 2) Automate updating gutenberg translations (void) Nov 16, 2019
@Tug Tug deleted the gutenberg/fix-i18n branch November 16, 2019 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix our i18n process
7 participants