URLs need to be (more) cacheable
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ratings and Reviews server |
Fix Released
|
Undecided
|
Michael Nelson | ||
software-center (Ubuntu) |
New
|
Undecided
|
Bug Description
When initial discussions for RNR started, we talked about caching of URLs for reviews etc., and I think it was for that reason that mvo/barry went with the long urls with the language, distro, series in the URL.
But bug 673948 indicates that we have the following additional complexities:
1) for reviews of a particular package we'll need to batch the results rather than returning all the results (note, pagination could also be added to the urls without too much trouble /en/ubuntu/
2) for review stats the client will need to request a set of stats based on what the client wants to display in the UI (ie. user searches for 'game') (eg. /en/ubuntu/
While we're there, it would be great to make the templates/tests independent of the urls (using reverse etc.)
Changed in rnr-server: | |
status: | New → In Progress |
assignee: | nobody → Michael Nelson (michael.nelson) |
Changed in rnr-server: | |
status: | In Progress → Fix Committed |
Changed in rnr-server: | |
milestone: | none → 11.01 |
status: | Fix Committed → Fix Released |
affects: | software-center → software-center (Ubuntu) |
While on the subject of URLs, I'm keen for the following additional changes - let me know if you disagree :)
1) Rename the app itself from reviewsapp ( a (django) application for reviews) -> appreviews ( (desktop) application reviews). moderations/ (we can use the language, origin, distroseries as *optional* filtering - currently the url requires them) moderations/ {id}/ (to which we can redirect after a moderation approval/rejection)
2) Change the root url from reviews -> appreviews
3) ReviewModeration plural resource would then be /appreviews/
4) ReviewModeration singular resource at /reviewsapp/