Skip to content

i18n: Use placeholder for minimum WordPress version notice#13487

Merged
aduth merged 2 commits intomasterfrom
update/version-notice-placeholder
Jan 24, 2019
Merged

i18n: Use placeholder for minimum WordPress version notice#13487
aduth merged 2 commits intomasterfrom
update/version-notice-placeholder

Conversation

@aduth
Copy link
Copy Markdown
Member

@aduth aduth commented Jan 24, 2019

See: #13370 (comment)

This pull request seeks to update the string used in the minimum required version notice to use placeholders. This is done to avoid the string changing for translators between revisions to the minimum required version.

Testing instructions:

Downgrade to an earlier version of WordPress while Gutenberg is active, and verify there are no regressions in the display of the version notice when Gutenberg self-deactivates.

@aduth aduth added the Internationalization (i18n) Issues or PRs related to internationalization efforts label Jan 24, 2019
@aduth aduth requested a review from swissspidy January 24, 2019 16:17
Co-Authored-By: aduth <andrew@andrewduthie.com>
@aduth aduth merged commit a99bc7d into master Jan 24, 2019
@aduth aduth deleted the update/version-notice-placeholder branch January 24, 2019 18:34
@aduth aduth added this to the 5.0 (Gutenberg) milestone Jan 24, 2019
youknowriad pushed a commit that referenced this pull request Mar 6, 2019
* i18n: Use placeholder for minimum WordPress version notice

* Collapse echo sprintf to printf

Co-Authored-By: aduth <andrew@andrewduthie.com>
youknowriad pushed a commit that referenced this pull request Mar 6, 2019
* i18n: Use placeholder for minimum WordPress version notice

* Collapse echo sprintf to printf

Co-Authored-By: aduth <andrew@andrewduthie.com>
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants