Ratings and Reviews server

Merge lp://staging/~fgallina/rnr-server/exclude-deleted-hidden-click-reviews into lp://staging/rnr-server

  1. exclude-deleted-hidden-click-reviews
  2. Merge into trunk
Proposed by Fabián Ezequiel Gallina
Status: Merged
Approved by: Fabián Ezequiel Gallina
Approved revision: 299
Merged at revision: 297
Proposed branch: lp://staging/~fgallina/rnr-server/exclude-deleted-hidden-click-reviews
Merge into: lp://staging/rnr-server
Prerequisite: lp://staging/~fgallina/rnr-server/delete-click-review-handler
Diff against target: 117 lines (+48/-7)
4 files modified
src/clickreviews/api/handlers.py (+4/-5)
src/clickreviews/models.py (+11/-2)
src/clickreviews/tests/test_handlers.py (+11/-0)
src/clickreviews/tests/test_models.py (+22/-0)
To merge this branch: bzr merge lp://staging/~fgallina/rnr-server/exclude-deleted-hidden-click-reviews
Reviewer Review Type Date Requested Status
Matias Bordese (community) Approve
Review via email: mp+252367@code.staging.launchpad.net

Commit message

clickreviews: Filter hidden and deleted reviews from handlers and stats

To post a comment you must log in.
Revision history for this message
Matias Bordese (matiasb) wrote :

LGTM

review: Approve
Revision history for this message
Ubuntu One Auto Pilot (otto-pilot) wrote :

There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions.

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