rnrclient doesn't allow stat-update urls
| 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://
http://
http://
http://
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 | #1 |
| Changed in rnr-server: | |
| status: | Fix Committed → Fix Released |
| Changed in rnr-server: | |
| milestone: | none → 10.12 |
this is working correctly