Repository navigation
Premium Analytics: add the Subscribers and Ads tabs to the customer preview - #52720
Merged
Merged
Conversation
…review The preview exposed Traffic and Insights. Subscribers and Ads join them for the soft launch. Ads is registered by the WordAds module and jetpack-mu-wpcom, so a site without WordAds still gets no Ads tab. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
dognose24
added
Enhancement
Contributor
|
Are you an Automattician? Please test your changes on all WordPress.com environments to help mitigate accidental explosions.
Interested in more tips and information?
|
github-actions
Bot
added
[Package] Premium Analytics
[Plugin] Jetpack
Contributor
|
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. |
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. 🤷 |
This was referenced Sep 23, 2026
dognose24
marked this pull request as ready for review
September 23, 2026 20:34
dognose24
added
[Status] Needs Review
github-actions
Bot
added
[Status] UI Changes
dognose24
added a commit
that referenced
this pull request
Sep 24, 2026
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>
dognose24
added a commit
that referenced
this pull request
Sep 24, 2026
…OOA7S-2208 lands (#52768) * Premium Analytics: take the Ads tab out of the customer preview 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> * Fold the changelog into the unreleased #52720 entries 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> --------- Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
louwie17
added a commit
that referenced
this pull request
Sep 28, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018rbRpLy4L7yKdf79uL4MFx
louwie17
added a commit
that referenced
this pull request
Sep 28, 2026
* 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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of the Stats v2 soft launch: the Ads tab (Jetpack Stats v2: Ads tab in Stats) and the Subscribers tab.
Proposed changes
Add
subscribersandadstoPREVIEW_SECTIONS, so a site in the Premium Analytics customer preview gets the Subscribers and Ads tabs alongside Traffic and Insights.This only widens the preview scope. Subscribers keeps its own availability gate. Whether the Ads section exists at all is still decided by its registrants since #52455: the WordAds module on the Jetpack plugin path and
jetpack-mu-wpcomon WordPress.com, each gated on WordAds being available and on the ad-reports capability. A site without WordAds sees no new tab (UNI-816 tracks promoting it there); a WordAds site in the preview now sees Ads without the a11n flag.Tests: the published scope now lists Subscribers, a plugin-registered
wordads/adssection lands in it, and Store stays out. One test assumed every preview tab is a package section; it now names the package's tabs, since Ads is not one.Related product discussion/links
Does this pull request change what data or activity we track or use?
No.
Testing instructions
On a WordPress.com Simple site opted into the preview with WordAds active (or the local Docker site with
zz-tmp-pa-wordads-mock.phpand the preview option on):?section=adsopens the tab, and/reports/earningsopens.jp test php packages/premium-analyticspasses.🤖 Generated with Claude Code