lp:~aaronp/rnr-server/delete-review

Branch merges

Propose for merging

Merged into lp:rnr-server at revision 183

Michael Nelson (community): Approve on 2011-05-06

Aaron Peachey (community): Needs Resubmitting on 2011-05-06

Diff: 502 lines (+294/-8)

10 files modified

src/reviewsapp/api/handlers.py (+9/-0)
src/reviewsapp/api/urls.py (+9/-1)
src/reviewsapp/forms.py (+3/-1)
src/reviewsapp/migrations/0010_add_review_date_deleted.py (+157/-0)
src/reviewsapp/models/reviews.py (+9/-1)
src/reviewsapp/tests/factory.py (+9/-4)
src/reviewsapp/tests/test_handlers.py (+58/-0)
src/reviewsapp/tests/test_rnrclient.py (+26/-0)
src/reviewsapp/tests/test_views.py (+12/-0)
src/reviewsapp/views/moderation.py (+2/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #778301: Reviewers should be able to delete own review

Low

Fix Released

Link a bug report

Related blueprints

187. By Aaron Peachey on 2011-05-06

- models/reviews.py: remove unused exceptions
- tests/test_handlers.py: tweak tests and add test
    to check stats are updated on delete

186. By Aaron Peachey on 2011-05-05

- test_rnrclient.py: add test for delete review in rnrclient
- api/handlers.py: return date_deleted in review response

185. By Aaron Peachey on 2011-05-05

add migration to bring in new date_deleted field to Review

184. By Aaron Peachey on 2011-05-05

* delete review functionality:
- modify to simplify handler in case of error
- ensure deleted reviews don't get moderated
- ensure user can submit review if one has been deleted
- update order of url parts
- add/modify test cases for above changes

183. By Aaron Peachey on 2011-05-02

add functionality to allow user to delete single review of their own (and tests)

182. By Anthony Lenton on 2011-04-21

[r=newz] Remove the default calls to action offered to the user in 404.html

181. By Anthony Lenton on 2011-04-21

Remove the hide_meta=True option when setting up our OOPSWare wsgi middleware.

180. By Anthony Lenton on 2011-04-20

Add a view to trigger an oops report.

179. By Łukasz Czyżykowski on 2011-04-19

Fix rnr client test which was hitting a 404 page instead instead of proper handler.

178. By Michael Nelson on 2011-04-12

[r=lukasz][bug=757695] Enable requests for stats across origins.