Merge lp://staging/~fgallina/rnr-server/edit-click-review-handler into lp://staging/rnr-server

Proposed by Fabián Ezequiel Gallina
Status: Merged
Approved by: Fabián Ezequiel Gallina
Approved revision: 296
Merged at revision: 295
Proposed branch: lp://staging/~fgallina/rnr-server/edit-click-review-handler
Merge into: lp://staging/rnr-server
Diff against target: 343 lines (+173/-15)
7 files modified
src/clickreviews/api/handlers.py (+17/-1)
src/clickreviews/api/urls.py (+3/-0)
src/clickreviews/forms.py (+22/-0)
src/clickreviews/models.py (+8/-2)
src/clickreviews/tests/test_forms.py (+60/-0)
src/clickreviews/tests/test_handlers.py (+45/-12)
src/clickreviews/tests/test_models.py (+18/-0)
To merge this branch: bzr merge lp://staging/~fgallina/rnr-server/edit-click-review-handler
Reviewer Review Type Date Requested Status
Natalia Bidart (community) Approve
Review via email: mp+242540@code.staging.launchpad.net

Commit message

clickreviews: Added review edit endpoint

PUT click/api/1.0/reviews/<review_id>/ with summary, review_text and
rating in request.data will modify a review provided it has no
moderations pending and it was created within the
`MODIFY_WINDOW_MINUTES` window.

To post a comment you must log in.
294. By Fabián Ezequiel Gallina

Merged trunk into edit-click-review-handler.

Revision history for this message
Natalia Bidart (nataliabidart) :
295. By Fabián Ezequiel Gallina

Fixes per review

296. By Fabián Ezequiel Gallina

Small cleanup

Revision history for this message
Natalia Bidart (nataliabidart) wrote :

Looks good!

review: Approve

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