Page MenuHomePhabricator
Create Task
Maniphest T87712

cxserver REST apis should have version number
Closed, ResolvedPublic1 Estimated Story Points
Actions

Description

Without version number, updating client and server has to be done at the same time. With version number we can for a short while support two versions of same api in cxserver side and update client side safely.

Details

Related Changes in Gerrit:
SubjectRepoBranchLines +/-
Use explicit cxserver REST API version number in configurationmediawiki/extensions/ContentTranslationmaster+1 -1
Provide version number in REST apismediawiki/services/cxservermaster+202 -170
Customize query in gerrit

Related Objects
Search...

Event Timeline

Nikerabbit created this task.Jan 27 2015, 11:27 PM
Nikerabbit raised the priority of this task from to Needs Triage.
Nikerabbit updated the task description. (Show Details)
Nikerabbit added a project: ContentTranslation-CXserver.
Nikerabbit subscribed.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJan 27 2015, 11:27 PM
Nikerabbit added a project: ContentTranslation.Feb 3 2015, 7:08 AM
santhosh triaged this task as High priority.Feb 4 2015, 4:00 AM
santhosh subscribed.
Amire80 subscribed.Feb 4 2015, 2:07 PM

Why is this high priority?

santhosh added a project: LE-Sprint-82.Feb 5 2015, 7:18 AM
In T87712#1014393, @Amire80 wrote:

Why is this high priority?

To prevent all clients of cxserver break when cxserver API change. Happened already in the last minute of production deployment,

santhosh renamed this task from cxserver rest apis should have version number to cxserver REST apis should have version number.Feb 5 2015, 7:20 AM
santhosh moved this task from Backlog to In Progress on the LE-Sprint-82 board.
santhosh set Security to None.
santhosh claimed this task.Feb 5 2015, 8:56 AM
santhosh moved this task from In Progress to In Review on the LE-Sprint-82 board.
gerritbot added a project: Patch-For-Review.Feb 5 2015, 8:56 AM
gerritbot subscribed.

Change 188764 had a related patch set uploaded (by Santhosh):
Provide version number in REST apis

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

Patch-For-Review

gerritbot added a comment.Feb 5 2015, 9:05 AM

Change 188767 had a related patch set uploaded (by Santhosh):
Use explicit cxserver REST API version number in configuration

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

Patch-For-Review

gerritbot added a comment.Feb 6 2015, 10:15 AM

Change 188764 merged by jenkins-bot:
Provide version number in REST apis

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

santhosh closed this task as Resolved.Feb 7 2015, 7:05 AM
Nikerabbit added a parent task: T87656: Do not wrap returned translation in a div.Feb 7 2015, 10:05 AM
gerritbot added a comment.Feb 13 2015, 4:30 AM

Change 188767 merged by jenkins-bot:
Use explicit cxserver REST API version number in configuration

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

• Jsahleen mentioned this in rECTX8e76a3a789b8: Use explicit cxserver REST API version number in configuration.Feb 13 2015, 4:30 AM
KartikMistry moved this task from In Review to Done on the LE-Sprint-82 board.Feb 17 2015, 3:55 AM
Pginer-WMF edited a custom field.Feb 24 2015, 2:21 PM
• Arrbee moved this task from Needs Triage to Long term on the ContentTranslation board.Mar 5 2015, 8:23 AM
Diffusion mentioned this in rMEXT7ae43ec55d4b: Updated mediawiki/extensions Project: mediawiki/extensions/ContentTranslation….Mar 11 2015, 6:26 AM
santhosh mentioned this in rGCXS3e847b3827b0: Provide version number in REST apis.Apr 8 2015, 4:12 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits