Closed
Bug 989181
Opened 11 years ago
Closed 9 years ago
Use CDN for app ratings responses
Categories
(Marketplace Graveyard :: Consumer Pages, defect, P3)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: cvan, Unassigned)
References
()
Details
(Keywords: perf, Whiteboard: [marketplace-transition])
As of bug 983815, we're using the CDN for API responses for the homepage, category, and search endpoints.
Because we lazy-load these in on the detail pages, and they take a few seconds on even fast wifi, let's use the CDN for the ratings endpoints too:
/api/v1/apps/rating/
https://marketplace.firefox.com/api/v1/apps/rating/?app=cupcakes-vs-veggies&lang=en-US®ion=us
Comment 1•11 years ago
|
||
More of a consumer pages bug (I suspect most of the changes will have to be made in fireplace)
Component: API → Consumer Pages
Updated•11 years ago
|
Blocks: tarako-marketplace
Updated•11 years ago
|
No longer blocks: tarako-marketplace
Updated•11 years ago
|
Priority: -- → P3
Updated•11 years ago
|
Assignee: nobody → mpillard
Comment 3•11 years ago
|
||
Like bug 991955 this is not trivial because user authentication is involved. I'm pretty sure we could rework some of that code to make sure we don't use user auth except when posting/editing, that would be a good start.
No longer blocks: 992365
Updated•11 years ago
|
Assignee: mpillard → nobody
Updated•10 years ago
|
Blocks: marketplace-perf
Comment 4•10 years ago
|
||
Why is user authentication involved here? Shouldn't the API response be the same no matter what and the choice to display a button or not be left up to the front end?
Comment 5•10 years ago
|
||
The API returns whether or not the user has already rated this app and whether or not he has already flagged each rating returned. The front-end then uses that info to figure out whether to display buttons or not.
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Whiteboard: [marketplace-transition]
You need to log in
before you can comment on or make changes to this bug.
Description
•