Skip to content

Conversation

@kean
Copy link
Contributor

@kean kean commented Jan 19, 2026

This initial PR adds an empty AdsTabView for sites that have WordAds module enabled.

Note: I opened CMM-1132: Add WordAds ads revenue stats to Jetpack app to improve who AdaptiveTabBar performs sizing if the content doesn't fit.

Screenshots

Screenshot 2026-01-19 at 3 09 43 PM Screenshot 2026-01-19 at 3 09 34 PM

Testing instructions

  • Enable the "Ads" feature flag
  • To enable, WordAds, as a site owner, got to “Jetpack / Monetize / Ads”
  • Verify that the new tab is shown (empty for now)

@kean kean added the Stats label Jan 19, 2026
@kean kean added this to the 26.7 milestone Jan 19, 2026
@kean kean marked this pull request as draft January 19, 2026 20:14
@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jan 19, 2026

App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number30485
VersionPR #25145
Bundle IDorg.wordpress.alpha
Commitb998d6f
Installation URL4ok4llqfgqir0
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jan 19, 2026

App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number30485
VersionPR #25145
Bundle IDcom.jetpack.alpha
Commitb998d6f
Installation URL187mqo3q99r60
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jan 19, 2026

🤖 Build Failure Analysis

This build has failures. Claude has analyzed them - check the build annotations for details.

@kean kean force-pushed the task/jetpack-stats-ads-tab branch from a38ad86 to c3b1812 Compare January 19, 2026 21:04
// MARK: - Helpers

/// Returns the current blog from SiteStatsInformation if available
private static func currentBlog() -> Blog? {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not ideal, but this is how this screen operates. I don't want to refactor it in the scope of this PR.

/// Enumeration that contains all of the Blog's available capabilities.
///
public enum Capability: String {
case ActivateWordAds = "activate_wordads"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are not using it yet, but I decided to model it anyway since I spent a bit of time discovering it.

@kean kean requested a review from crazytonyli January 19, 2026 21:06
@kean kean marked this pull request as ready for review January 19, 2026 21:06
@kean kean force-pushed the task/jetpack-stats-ads-tab branch from 08a5b43 to b998d6f Compare January 19, 2026 22:25
@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants