Merge lp://staging/~mvo/rnr-server/compat-with-older-clients into lp://staging/rnr-server

Proposed by Michael Vogt
Status: Rejected
Rejected by: Michael Vogt
Proposed branch: lp://staging/~mvo/rnr-server/compat-with-older-clients
Merge into: lp://staging/rnr-server
Diff against target: 54 lines (+33/-0)
2 files modified
src/reviewsapp/api/urls.py (+11/-0)
src/reviewsapp/tests/test_handlers.py (+22/-0)
To merge this branch: bzr merge lp://staging/~mvo/rnr-server/compat-with-older-clients
Reviewer Review Type Date Requested Status
Ratings and Reviews Developers Pending
Review via email: mp+47650@code.staging.launchpad.net

Description of the change

This branch provides a simple mapping to the previous url schema for getting reviews. This way we can support the old and the new client for some time in parallel. Eventually this compat code can be dropped, but it will make deployment easier this way.

To post a comment you must log in.
Revision history for this message
Danny Tamez (zematynnad) wrote :

Michael - I think Anthony sneaked his in right before you: https://code.launchpad.net/~elachuni/rnr-server/old-api-support/+merge/47644
.

Revision history for this message
Michael Vogt (mvo) wrote :

Indeed - and his code is nicer.

Unmerged revisions

114. By Michael Vogt

add compatibility for rnr-server clients that do not yet send "version" as part of the "get-reviews" API

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: