rnrclient doesn't allow stat-update urls

Bug #688114 reported by Michael Nelson on 2010-12-09
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ratings and Reviews server
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
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  Edit
Everyone can see this information.

Other bug subscribers