- 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.
Description
Description
Details
Details
Related Changes in Gerrit:
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| summary: mark api_urls, title, displaytitle as deprecated | mediawiki/services/mobileapps | master | +10 -3 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | • bearND | T247991 Remove summary api_urls field | |||
| Resolved | • bearND | T251924 Evaluate if we can remove the api_urls field in the summary output | |||
| Resolved | • bearND | T251959 Mark fields in summary as deprecated |
Event Timeline
Comment Actions
Change 601895 had a related patch set uploaded (by BearND; owner: BearND):
[mediawiki/services/mobileapps@master] summary: mark api_urls, title, displaytitle as deprecated
Comment Actions
Change 601895 merged by jenkins-bot:
[mediawiki/services/mobileapps@master] summary: mark api_urls, title, displaytitle as deprecated
Comment Actions
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?