Code review comment for lp:~aaronp/rnr-server/delete-review

Revision history for this message
Aaron Peachey (aaronp) wrote :

One more thing for now. Updated and tested rnrclient (diff below).
Added a test for the rnrclient change and added date_deleted to the fields in the show review handler so it gets returned when a review is successfully deleted (I don't know why this worked, but I took a guess and it did!)

=== modified file 'rnrclient.py'
--- rnrclient.py 2011-03-17 09:17:26 +0000
+++ rnrclient.py 2011-05-05 14:10:04 +0000
@@ -167,3 +167,10 @@

         return self._get('usefulness/', args=data,
             scheme=PUBLIC_API_SCHEME)
+
+ @validate('review_id', int)
+ @returns_json
+ def delete_review(self, review_id):
+ """Delete a review"""
+ return self._post('/reviews/delete/%s/' % review_id, data = {},
+ scheme=AUTHENTICATED_API_SCHEME)

« Back to merge proposal