Ratings and Reviews server

Merge lp://staging/~fgallina/rnr-server/moderation-endpoints-tweaks into lp://staging/rnr-server

  1. moderation-endpoints-tweaks
  2. Merge into trunk
Proposed by Fabián Ezequiel Gallina
Status: Merged
Approved by: Ricardo Kirkner
Approved revision: 291
Merged at revision: 289
Proposed branch: lp://staging/~fgallina/rnr-server/moderation-endpoints-tweaks
Merge into: lp://staging/rnr-server
Diff against target: 159 lines (+56/-20)
2 files modified
src/clickreviews/api/handlers.py (+15/-20)
src/clickreviews/tests/test_handlers.py (+41/-0)
To merge this branch: bzr merge lp://staging/~fgallina/rnr-server/moderation-endpoints-tweaks
Reviewer Review Type Date Requested Status
Ricardo Kirkner (community) Approve
Review via email: mp+238553@code.staging.launchpad.net

Commit message

clickreviews: Enhacements to moderation endpoints

1. Sort moderation data by number of flags and creation date
2. Expand full user data in `ClickPackageReviewModerationHandler`

To post a comment you must log in.
Revision history for this message
Ricardo Kirkner (ricardokirkner) wrote :

Looks good overall, but a few questions pending

lp://staging/~fgallina/rnr-server/moderation-endpoints-tweaks updated
291. By Fabián Ezequiel Gallina

Fixes per review

Revision history for this message
Ricardo Kirkner (ricardokirkner) wrote :

LGTM

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