Page MenuHomePhabricator

Mark fields in summary as deprecated
Closed, ResolvedPublic

Description

  • Consider marking the api_urls field as deprecated. (This could be done by adding deprecated: true right where api_urls: is defined in the spec.yaml).
  • While we're at it we could also mark some of the duplicative title fields as deprecated. The ones in titles should be used instead. (See https://en.wikipedia.org/api/rest_v1/page/summary/The_Rebel_(book) for some good examples of the differences.)
    • title
    • displaytitle
  • We should update the summary spec in RESTBase to make sure we have everything that lives in the version in the mobileapps repo reflected in RB as well.

Event Timeline

LGoto triaged this task as Medium priority.May 6 2020, 3:36 PM
LGoto moved this task from Needs triage to Upcoming on the Product-Infrastructure-Team-Backlog board.

Change 601895 had a related patch set uploaded (by BearND; owner: BearND):
[mediawiki/services/mobileapps@master] summary: mark api_urls, title, displaytitle as deprecated

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

Change 601895 merged by jenkins-bot:
[mediawiki/services/mobileapps@master] summary: mark api_urls, title, displaytitle as deprecated

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

Looks like this RB change is merged but not deployed yet. I still see the old summary version 1.3.7 and the fields are not marked as deprecated in https://en.wikipedia.org/api/rest_v1/#/Page%20content/get_page_summary__title_. @Pchelolo, would you let us know when this gets deployed?

RB changes were deployed today.