Page MenuHomePhabricator

Generalise PageList to take Page as parameter
Closed, ResolvedPublic

Description

Whilst working with PageList I was confused by the inconsistencies between the objects in PageList#defaults.pages and the Page.options

I think we should make defaults.page an array of pages rather than a list of titles.

Event Timeline

Jdlrobson raised the priority of this task from to Needs Triage.
Jdlrobson updated the task description. (Show Details)
Jdlrobson added a project: Readers-Web-Backlog.
Jdlrobson moved this task to Triaged but Future on the Readers-Web-Backlog board.
Jdlrobson added a subscriber: Jdlrobson.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 29 2015, 10:13 PM

Change 203266 had a related patch set uploaded (by Jdlrobson):
Hygiene: Make PageList take an array of pages

https://gerrit.wikimedia.org/r/203266

Change 203266 merged by jenkins-bot:
Hygiene: Make PageList take an array of pages

https://gerrit.wikimedia.org/r/203266

Jdlrobson closed this task as Resolved.Jun 10 2015, 8:05 PM