wordpress-mobile / WordPress-Android

WordPress for Android
http://android.wordpress.org
GNU General Public License v2.0
2.92k stars 1.3k forks source link

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

Open staskus opened 2 weeks ago

staskus commented 2 weeks ago

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 commented 2 weeks ago

Thanks for reporting! đź‘Ť

staskus commented 2 weeks ago

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.