Repository navigation
Premium Analytics: Take the Ads tab out of the customer preview until WOOA7S-2208 lands - #52768
Conversation
Gary's call on the soft-launch thread: hold the Ads tab until it shows only on sites that use WordAds (WOOA7S-2208). Subscribers stays in the preview. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
|
Are you an Automattician? Please test your changes on all WordPress.com environments to help mitigate accidental explosions.
Interested in more tips and information?
|
|
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! Jetpack plugin: The Jetpack plugin has different release cadences depending on the platform:
If you have any questions about the release process, please ask in the #jetpack-releases channel on Slack. Wpcomsh plugin:
If you have any questions about the release process, please ask in the #jetpack-releases channel on Slack. Premium Analytics plugin: No scheduled milestone found for this plugin. If you have any questions about the release process, please ask in the #jetpack-releases channel on Slack. |
Both would ship in the same release and contradict each other, so the Subscribers-and-Ads entries now say Subscribers only. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Code Coverage SummaryThis PR did not change code coverage! That could be good or bad, depending on the situation. Everything covered before, and still is? Great! Nothing was covered before? Not so great. 🤷 |
* Premium Analytics: Put the Ads tab back in the customer preview This reverts commit 6859a54 (#52768). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018rbRpLy4L7yKdf79uL4MFx * Add new changelog entries instead of editing the #52720 ones Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018rbRpLy4L7yKdf79uL4MFx --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Part of WOOA7S-2208
Proposed changes
Take
adsback out ofPREVIEW_SECTIONS, so a site in the Premium Analytics customer preview sees Traffic, Insights and Subscribers, as #52720 left Subscribers. Gary's call on the soft-launch thread: hold the Ads tab until it shows only on sites that actually use WordAds (WOOA7S-2208, #52766), because today every Premium-and-up WordPress.com site gets it, with empty earnings if ads were never switched on.Nothing else changes. Sites running the dashboard through the override sticker or the a11n flag keep every tab, including Ads, as before. The section registration and its gates are untouched; #52766 fixes those, and
adsgoes back into the preview once it lands.The changelog is folded into the still-unreleased #52720 entries, which now say Subscribers only, so one release does not announce the Ads tab and its removal side by side.
Related product discussion/links
Does this pull request change what data or activity we track or use?
No.
Testing instructions
jp test php packages/premium-analyticspasses.?section=adsfalls back to Traffic.jetpack-premium-analyticsoverride sticker, the Ads tab is still there.🤖 Generated with Claude Code