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

Remove 4 instances of 'gutenberg' text domain from WordPress core #37229

Merged
merged 7 commits into from
Dec 13, 2021

Conversation

kebbet
Copy link
Contributor

@kebbet kebbet commented Dec 8, 2021

Closes #37228

Description

How has this been tested?

Screenshots

Types of changes

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • I've tested my changes with keyboard and screen readers.
  • My code has proper inline documentation.
  • I've included developer documentation if appropriate.
  • I've updated all React Native files affected by any refactorings/renamings in this PR (please manually search all *.native.js files for terms that need renaming or removal).

Use sprintf() to combine the string with the $label.
Use sprintf() to combine the string with the $label.
@kebbet kebbet changed the title Remove text domain Remove 4 instances of 'gutenberg' text domain from WordPress core Dec 8, 2021
@noisysocks noisysocks added [Type] Bug An existing feature does not function as intended Internationalization (i18n) Issues or PRs related to internationalization efforts labels Dec 8, 2021
@noisysocks noisysocks added this to 👀 Needs review in WordPress 5.9 Must-Haves via automation Dec 8, 2021
@noisysocks noisysocks added the Backport to WP 6.6 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Dec 8, 2021
Copy link
Member

@Mamaduka Mamaduka left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this, @kebbet.

Changes look good, I just left a small comment, but it's not a blocker.

@Mamaduka
Copy link
Member

Hi, @ocean90

Can we merge this PR?

@aristath aristath merged commit 3660a92 into WordPress:trunk Dec 13, 2021
WordPress 5.9 Must-Haves automation moved this from 👀 Needs review to ✅ Done Dec 13, 2021
@github-actions github-actions bot added this to the Gutenberg 12.2 milestone Dec 13, 2021
@aristath
Copy link
Member

All comments were addressed. I reviewed the commit and it looks good, fixing the issue at hand. No failing tests, so I went ahead and merged it 👍

@noisysocks noisysocks removed the Backport to WP 6.6 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Dec 14, 2021
noisysocks pushed a commit that referenced this pull request Dec 14, 2021
…7229)

* Remove text domain, set string once.

Use sprintf() to combine the string with the $label.

* Remove text domain, set string once.

Use sprintf() to combine the string with the $label.

* Update variable name and translator text.

Thanks for feedback!

* Update variable name and translator comment. 

Thanks for feedback!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Internationalization (i18n) Issues or PRs related to internationalization efforts [Type] Bug An existing feature does not function as intended
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Gutenberg textdomain in WordPress 5.9 beta 2.
5 participants