We're currently retrieving a set number of edits from the API and then filtering them down before displaying them to the user in the Personal Dashboard. This leaves the door open to not finding enough edits to display. If we don't find as many as we anticipate (at least 5, currently), we could be going back to continue the API call to find another set of edits to filter.
This should have a sensible cap so we don't endlessly make API calls on e.g. test or local wikis.
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Open | None | T422599 WE1.11: Improve Review Changes | |||
| Open | None | T409059 WE1.11.1: Improve Review Changes module / Surface noteworthy edits | |||
| Resolved | Samwalton9-WMF | T420726 Denote whether a change has already been opened by this user in the PersonalDashboard Review Changes feed | |||
| In Progress | Scardenasmolinar | T422607 Define and add Pinia storage for PersonalDashboard Review Changes module | |||
| Open | Kgraessle | T419444 If we don't find enough edits to display to the user, extend the API call to another group of edits for filtering |
I was chatting in RTL that this may make more sense to implement as a follow on to the pinia store refactor in T422607: Define and add Pinia storage for PersonalDashboard Review Changes module.
Change #1272742 had a related patch set uploaded (by Kgraessle; author: Kgraessle):
[mediawiki/extensions/PersonalDashboard@master] If we don't find enough edits to display to the user, extend the API call to another group of edits for filtering
Change #1272760 had a related patch set uploaded (by Kgraessle; author: Kgraessle):
[mediawiki/extensions/PersonalDashboard@master] If we don't find enough edits to display to the user, extend the API call to another group of edits for filtering
Change #1272742 abandoned by Kgraessle:
[mediawiki/extensions/PersonalDashboard@master] If we don't find enough edits to display to the user, extend the API call to another group of edits for filtering
Reason:
Pushed branch incorrectly
Change #1272761 had a related patch set uploaded (by Kgraessle; author: Kgraessle):
[mediawiki/extensions/PersonalDashboard@master] If we don't find enough edits to display to the user, extend the API call to another group of edits for filtering
Change #1272760 abandoned by Kgraessle:
[mediawiki/extensions/PersonalDashboard@master] If we don't find enough edits to display to the user, extend the API call to another group of edits for filtering
Reason:
Dupe
Change #1272761 merged by jenkins-bot:
[mediawiki/extensions/PersonalDashboard@master] If we don't find enough edits to display to the user, extend the API call to another group of edits for filtering