-
Notifications
You must be signed in to change notification settings - Fork 57
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
Enable page templates #1987
Enable page templates #1987
Conversation
@@ -3,9 +3,16 @@ import Foundation | |||
private func dummy() { | |||
_ = NSLocalizedString("'%s' isn't yet supported on WordPress for Android", comment: "") | |||
_ = NSLocalizedString("'%s' isn't yet supported on WordPress for iOS", comment: "") | |||
_ = NSLocalizedString("(555)555-1234", comment: "translators: sample content for \"Contact\" page template") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it expected that we don't see similar changes for Android?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They're there in bundle/android/strings.xml, but GitHub has the bad habit of not showing some diffs by default, so it was easy to miss
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thank you
This PR removes the
__mobileEnablePageTemplates
feature flags and enables page templates by default.Fixes #1962
gutenberg-mobile PR: #1987
gutenberg PR: WordPress/gutenberg#20718
WPiOS PR: wordpress-mobile/WordPress-iOS#13595
WPAndroid PR: wordpress-mobile/WordPress-Android#11409
To test:
From either WPiOS or WPAndroid, create a new page and verify that the template picker is showing
PR submission checklist:
RELEASE-NOTES.txt
if necessary.