Display ratings and reviews for each app

Bug #788207 reported by Anthony Lenton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu Apps Directory
Fix Released
Medium
Michael Nelson

Bug Description

Ideally:
 - reviews will be pulled in using ajax, to reduce page load times
 - a page with reviews included statically would be available and linked in, for better seo.
 - the look of reviews would be identical to what you get in software-center itself.

This can probably be split into several bugs

Related branches

Changed in ubuntu-webcatalog:
status: New → Confirmed
importance: Undecided → Medium
tags: added: kb-feature sp-1
Revision history for this message
Michael Nelson (michael.nelson) wrote :

I'm assuming that the first part of this should be importing the review stats on a schedule, as that'll give us the ratings and historgram etc. for each app without requiring any ajax.

If this *isn't* the case, let me know :)

Revision history for this message
Anthony Lenton (elachuni) wrote :

@noodles,
Yep, pulling the ratings and histograms on a schedule sounds fine. As for the actual reviews for an app, I imagine those would be pulled in on demand and cached, for a certain time.
Possibly storing these in the DB also, along with the data for the app, with an expiration time? I'm not sure relying on something like httplib2's caching would be enough.

Revision history for this message
Michael Nelson (michael.nelson) wrote :

Just a screenshot showing the non-js view for displaying reviews - the reviews of which will be identical when displayed on the detail page via js.

Changed in ubuntu-webcatalog:
status: Confirmed → Fix Committed
tags: added: sp-4
removed: sp-1
Changed in ubuntu-webcatalog:
status: Fix Committed → Fix Released
assignee: nobody → Michael Nelson (michael.nelson)
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.