Closed
Bug 1098549
Opened 11 years ago
Closed 11 years ago
model cache apps on collection landing pages
Categories
(Marketplace Graveyard :: Consumer Pages, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
2014-11-25
People
(Reporter: kngo, Assigned: kngo)
Details
(Whiteboard: [perf][repoman])
It was about a month ago I figured out we were deserializing entire app objects in collection detail pages. So we should model cache those apps so that clicking on the apps doesn't fire an API request.
|
Assignee | |
Updated•11 years ago
|
Assignee: nobody → kngo
Priority: -- → P2
Whiteboard: [perf][repoman]
|
|
Assignee | |
Comment 1•11 years ago
|
||
https://github.com/mozilla/fireplace/pull/809
STR:
1. Open your console.
2. Go to the New, Popular, Recommended, Feed Collection Detail, Feed Brand Detail, Feed Shelf detail pages.
Expected:
On each load of the pages, you should see something about apps being model-cached. Like:
[model] Stored Twitter as app
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2014-11-25
|
||
Comment 2•11 years ago
|
||
Verified as fixed in FF36(Win7) in marketplace-dev.allizom.org
In each page mentioned in the previous comment the [model] Stored "appName" as app is displayed in the console.
Closing bug.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•