wordpress-mobile / wordpress-mobile/WordPress-iOS
Replace stats/follows with v2/stats/subscribers in People screen
Open
Nobody has claimed this yet.
[Type] Task
Stats
- Dominant language
- Swift
- Stars
- 3.9k
- Forks
- 1.2k
- Avg merge
- 23h 51m
- Merged PRs (30d)
- 58
Description
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)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Search the repository for v1.1/sites/{site_id}/follows and trace the request from Stats Subscribers' View More action into the People view. Update that request to use v2/sites/{site_id}/subscribers, then verify that a non-admin editor can load the full Subscriber list.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- ios, swift
- Domain
- api, mobile
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100