Write: expand Write editor eligibility during gradual rollout#48730
Write: expand Write editor eligibility during gradual rollout#48730allilevine wants to merge 3 commits into
Conversation
Expand Write editor loading to also trigger for a12s, in addition to sites with the wpcom-write-editor blog sticker. Part of RSM-2870. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Thank you for your PR! When contributing to Jetpack, we have a few suggestions that can help us test and review your patch:
This comment will be updated as you work on your PR and make changes. If you think that some of those checks are not needed for your PR, please explain why you think so. Thanks for cooperation 🤖 Follow this PR Review Process:
If you have questions about anything, reach out in #jetpack-developers for guidance! |
|
Are you an Automattician? Please test your changes on all WordPress.com environments to help mitigate accidental explosions.
Interested in more tips and information?
|
Code Coverage SummaryCoverage changed in 1 file.
|
is_automattician() is only available on Simple; add A8C_PROXIED_REQUEST check so Atomic sites also enable the Write editor for a12s. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
kbrown9
left a comment
There was a problem hiding this comment.
The logic looks good to me! I did a quick test with an a11n account with simple/atomic and stickered/non-stickered sites, and I was able to access the Write editor on all of them.
Related to RSM-2870
Proposed changes
wpcom-write-editorblog sticker.Related product discussion/links
Does this pull request change what data or activity we track or use?
No.
Testing instructions
wpcom-write-editorsticker, verify the Write editor is accessible at/wp-admin/admin.php?page=writefor qualifying users.