Page MenuHomePhabricator

Should be possible to filter meta=readinglists by reading list id
Closed, ResolvedPublic


When rendering a reading list I want to be able to render all pages in the list along with the title and description of the list.
It seems the meta=readinglists API parameter does not allow for filtering by id

For a list with id 1 it might look like this:


Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

@Jdlrobson we should talk about your workflow so we can understand it.

We assumed that you would fetch the list of reading lists (with metadata) and show that to the user. Then they could tap a readinglist and you would then show the entries.

It sounds like you will have a reading list id, but none of the meta data... can you explain why and how you are retaining the list id, but aren’t able to retain the other metadata about the list?

It just may be we misunderstood how web would interact with the API.

The web has no concept of state, so when I tap a reading list I am navigated to "/wiki/Special:ReadingLists/Jdlrobson/2/My list name" (the list with id 2). I want to render the description and title of this as well as the list of items. I could of course just make this a single page app, but this is not very web-like and it's nice to be able to bookmark a reading list or find it in the address bar by typing its name (which is in the URL).

There is no way to get meta data for a single list - right now I'm working around the problem by requesting all lists (which is fine up until the point you have more than 50 lists).

Change 439519 had a related patch set uploaded (by Gergő Tisza; owner: Gergő Tisza):
[mediawiki/extensions/ReadingLists@master] Add list parameter to meta=readinglists

Change 439519 merged by jenkins-bot:
[mediawiki/extensions/ReadingLists@master] Add list parameter to meta=readinglists

Tgr closed this task as Resolved.Jun 13 2018, 7:37 PM
Tgr claimed this task.
Vvjjkkii renamed this task from Should be possible to filter meta=readinglists by reading list id to 4pcaaaaaaa.Jul 1 2018, 1:10 AM
Vvjjkkii reopened this task as Open.
Vvjjkkii removed Tgr as the assignee of this task.
Vvjjkkii triaged this task as High priority.
Vvjjkkii updated the task description. (Show Details)
Vvjjkkii edited subscribers, added: Tgr; removed: gerritbot, Aklapper.
Esteban16 renamed this task from 4pcaaaaaaa to Should be possible to filter meta=readinglists by reading list id.Jul 1 2018, 2:31 AM
bearND closed this task as Resolved.Jul 1 2018, 4:08 AM
bearND assigned this task to Tgr.
bearND raised the priority of this task from High to Needs Triage.
bearND updated the task description. (Show Details)