Page MenuHomePhabricator

Fix technical debt in ReadingLists extension
Open, LowPublic


I have left various ReadingLists issues unaddressed as they don't matter much as long as the extension gets limited use, and working on them would block other work that blocks the apps from using reading lists. These should be fixed before the service gets serious load (so, Q3-ish at the latest).

  • prevent adding/removing entries from deleted lists?
  • use non-offset-based continuation where possible (improves performance and robustness)
  • remove /setup, setup automatically on first write?
  • better names for things? (esp. setup/teardown)
  • pin down the semantics of the metadata fields (color, image, icon). And for that matter, the fields themselves. T180092
  • rethink where/how to return list/entry order
  • from T182053: getListsByDateUpdated has two dedicated indexes, but it could probably use the same indexes as getAllLists with some trickery
  • from T182053: getListEntries filesorts when asked for entries of multiple lists; this could probably be avoided by sorting by list ID first.

Event Timeline

Tgr created this task.Jul 27 2017, 10:33 PM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Tgr updated the task description. (Show Details)Nov 8 2017, 12:02 AM
Tgr updated the task description. (Show Details)Nov 13 2017, 1:16 AM
Tgr moved this task from Backlog to Coming soon on the Reading List Service board.Dec 6 2017, 12:40 AM
Tgr updated the task description. (Show Details)Dec 6 2017, 12:57 AM
Tgr moved this task from Coming soon to Backlog on the Reading List Service board.
Jhernandez triaged this task as Low priority.Mar 20 2019, 3:53 PM