rnrclient doesn't allow stat-update urls

Bug #688114 reported by Michael Nelson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ratings and Reviews server
Fix Released
Low
Michael Nelson

Bug Description

The current rnrclient only allows review_stats() (ie. pulling the complete set of stats). It should provide options to pull just the last 1, 3 or 7 days of updates too.

Also the get_reviews() method doesn't yet support the page kwarg (the handler enforces standard pagination of results for caching).

Notes to QA: You should be able to test this on the vps using the following urls:
  http://184.82.116.62/reviews/api/1.0/review-stats/ - full set of apps
  http://184.82.116.62/reviews/api/1.0/review-stats/updates-last-1-days/ - Changes from the last day
  http://184.82.116.62/reviews/api/1.0/review-stats/updates-last-3-days/ - Same for last 3 days
  http://184.82.116.62/reviews/api/1.0/review-stats/updates-last-7-days/ - Same for last 7 days
You can't request an arbitrary amount of days, just 1, 3 or 7 (or the full set). After rating a new app the modified rating should appear on all four urls.

description: updated
tags: added: kb-defect
Changed in rnr-server:
assignee: nobody → Michael Nelson (michael.nelson)
Changed in rnr-server:
importance: Undecided → Low
status: New → In Progress
tags: added: proj-rnr-10.12
Changed in rnr-server:
status: In Progress → Fix Committed
Changed in rnr-server:
status: Fix Committed → In Progress
Changed in rnr-server:
status: In Progress → Fix Committed
description: updated
Revision history for this message
Dave Morley (davmor2) wrote :

this is working correctly

Changed in rnr-server:
status: Fix Committed → Fix Released
Changed in rnr-server:
milestone: none → 10.12
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.