Skip to content

Stats Review: Add Resources (P1) #24713

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: review_base/feature_jetpack-stats
Choose a base branch
from

Conversation

kean
Copy link
Contributor

@kean kean commented Aug 5, 2025

  • 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
Copy link
Collaborator

dangermattic commented Aug 5, 2025

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

Copy link

sonarqubecloud bot commented Aug 5, 2025

@kean kean added the Stats label Aug 5, 2025
@kean kean added this to the Pending milestone Aug 5, 2025
@wpmobilebot
Copy link
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
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