Skip to content

Stats Review: Add Resources (P1) - #24713

Closed
kean wants to merge 1 commit into
review_base/feature_jetpack-statsfrom
review/stats-mocks
Closed

Stats Review: Add Resources (P1)#24713
kean wants to merge 1 commit into
review_base/feature_jetpack-statsfrom
review/stats-mocks

Conversation

@kean

@kean kean commented Aug 5, 2025

Copy link
Copy Markdown
Contributor
  • Add mock data (anonymized, AI-generated)
  • Add mock avatars (AI-generated)

The mocks are used in SwiftUI previews and in the debug builds to showcase all the Stats features. There are included in the release builds because there is no simple way to exclude them in SwiftPM. The impact is ~200KB, so I didn't thought it was worth spending time and increase complexity over 200KB.

@kean kean mentioned this pull request Aug 5, 2025
14 tasks
@dangermattic

dangermattic commented Aug 5, 2025

Copy link
Copy Markdown
Collaborator
1 Warning
⚠️ This PR is larger than 500 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.

Generated by 🚫 Danger

@sonarqubecloud

sonarqubecloud Bot commented Aug 5, 2025

Copy link
Copy Markdown

@kean kean added the Stats label Aug 5, 2025
@kean kean added this to the Pending milestone Aug 5, 2025
@wpmobilebot

Copy link
Copy Markdown
Contributor
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 Number28391
VersionPR #24713
Bundle IDorg.wordpress.alpha
Commitcf712aa
Installation URL6kff8nba87fi8
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot

Copy link
Copy Markdown
Contributor
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 Number28391
VersionPR #24713
Bundle IDcom.jetpack.alpha
Commitcf712aa
Installation URL7sbrnt0jah480
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

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.

3 participants