Skip to content
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

Replace stats/follows with v2/stats/subscribers in People screen #23111

Open
staskus opened this issue Apr 29, 2024 · 2 comments
Open

Replace stats/follows with v2/stats/subscribers in People screen #23111

staskus opened this issue Apr 29, 2024 · 2 comments

Comments

@staskus
Copy link
Contributor

staskus commented Apr 29, 2024

The current v1.1/sites/{site_id}/follows endpoint we use doesn't work for non-admins, but the new v2/sites/{site_id}/subscribers one does work. So until we update it, editors who go to Stats Subscribers in the app and tap View More won't be able to load the full Subscriber list (aka. the People view)

@dangermattic
Copy link
Collaborator

Thanks for reporting! 👍

@staskus staskus self-assigned this May 1, 2024
@staskus
Copy link
Contributor Author

staskus commented May 2, 2024

Putting out of scope, due to findings on pdDOJh-2RW-p2:

The problem is that, due to how the data is organized in different databases, we deliver the features in different ways depending on the number of subscribers of your site. Let’s see what those differences are:
We can aggregate data for small sites, and we call this endpoint to return the subscriber list: /wpcom/v2/sites/[blog-id]/subscribers. We cannot aggregate data for big sites, so we need to pass a subscriber type all the time, using this endpoint instead: /wpcom/v2/sites/[blog-id]/subscribers_by_user_type.

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

No branches or pull requests

2 participants