Merge lp:~sil/rnr-server/clickreview-api-more-parameters into lp:rnr-server
Proposed by
Stuart Langridge
Status: | Rejected |
---|---|
Rejected by: | Natalia Bidart |
Proposed branch: | lp:~sil/rnr-server/clickreview-api-more-parameters |
Merge into: | lp:rnr-server |
Diff against target: |
125 lines (+72/-6) 4 files modified
src/clickreviews/admin.py (+14/-0) src/clickreviews/api/handlers.py (+10/-3) src/clickreviews/forms.py (+11/-1) src/clickreviews/tests/test_handlers.py (+37/-2) |
To merge this branch: | bzr merge lp:~sil/rnr-server/clickreview-api-more-parameters |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ratings and Reviews Developers | Pending | ||
Review via email: mp+245958@code.launchpad.net |
Description of the change
Allow the click reviews API to be queried by package_name (as currently) OR reviewer_username, with an eye to possibly broadening it still further in the future.
To post a comment you must log in.
Unmerged revisions
- 293. By Stuart Langridge
-
Allow the click reviews API to be queried by package_name (as currently) OR reviewer_username
Hi!
There are some tests missing, for example for the changes in the forms.py and admin.py. Also the MP looks quite old, setting MP as Rejected to reduce numbers of landing candidates in @reviewlist. Change status again if this MP is still current.
Thanks!