Premium Analytics: add Stats email breakdown endpoints - #49899
Conversation
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 comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
22ae2c6 to
2e258aa
Compare
|
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! |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Review-cycle audit: latest invited-AI feedback is non-blocking only, there are no inline review threads open, and GitHub reports no required checks for this branch. PR is non-draft and labeled |
5d997ce to
c173ce0
Compare
c173ce0 to
fa5a920
Compare
The all-time email opens/clicks breakdown endpoints are keyed solely by post ID and breakdown dimension; they take no period/date query params (those belong to the separate timeline endpoint). Drop the misleading StatsQueryParams bag from the queries and hooks, gate the queries on a valid post ID like the sibling post-stats query, publicly export StatsEmailBreakdownItem alongside the other Stats item types, and add query-layer coverage for the request paths, keys, and post-ID guard.
Fixes #
Proposed changes
Related product discussion/links
Does this pull request change what data or activity we track or use?
No. This only adds typed client data/query integration for an existing Stats API endpoint.
Testing instructions