-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[Jetpack Content Migration Flow] Enable feature flags #19759
Conversation
…ion, prevent notification dupes in WP.
You can test the changes in WordPress from this Pull Request by:
|
You can test the changes in Jetpack from this Pull Request by:
|
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.
ssndon
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
Noting that this PR doesn't include re-enabling remote feature flag functionality for the "Disable WordPress Notifications" project - it's just the local flag. See #19761. |
Description
This PR enables the following feature flags for the Content Migration Flow project:
jetpackPoweredBottomSheet
contentMigration
jetpackMigrationPreventDuplicateNotifications
Testing
To test, follow the project's test plan.
Internal ref: pcdRpT-1i3-p2
Regression Notes
Potential unintended areas of impact
What I did to test those areas of impact (or what existing automated tests I relied on)
What automated tests I added (or what prevented me from doing so)
PR submission checklist:
RELEASE-NOTES.txt
if necessary.