Merge lp://qastaging/~chipaca/unity-lens-video/handle-upcoming-server-changes into lp://qastaging/unity-lens-video/remote-videos-scope-trunk
Proposed by
John Lenton
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Merged at revision: | 38 | ||||
| Proposed branch: | lp://qastaging/~chipaca/unity-lens-video/handle-upcoming-server-changes | ||||
| Merge into: | lp://qastaging/unity-lens-video/remote-videos-scope-trunk | ||||
| Diff against target: |
30 lines (+6/-2) 1 file modified
src/unity-scope-video-remote (+6/-2) |
||||
| To merge this branch: | bzr merge lp://qastaging/~chipaca/unity-lens-video/handle-upcoming-server-changes | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| David Callé | Pending | ||
|
Review via email:
|
|||
Description of the change
In developing the video search API it has become obvious that we need to make an incompatible change to the results we return, in order to accommodate further needed changes.
Both /sources/ and /search/ currently return a list; this would change to return a dictionary, with the current list being under the 'results' key.
This change is going to happen "soon" (as soon as this branch is released and the new user agent is what's reaching the server). This branch makes the remote videos scope understand both the current list result and the future dictionary result. Thus when the switch on the server happens, everything continues to work.
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
