Closed
Bug 995418
Opened 11 years ago
Closed 11 years ago
[tarako] Move indexing code, app install code out of views/search.js
Categories
(Marketplace Graveyard :: Consumer Pages, defect, P1)
Tracking
(Not tracked)
RESOLVED
FIXED
2014-04-15
People
(Reporter: mat, Assigned: kngo)
References
Details
(Keywords: perf)
I copy/pasted the minimum I needed in detail.js, but really we should refactor search.js and move at least all the app install and index code out of it.
This will unbreak app install for details page.
Comment 1•11 years ago
|
||
The indexing code should allow for custom index configuration to be passed, and should return the appropriate promises allowing for it to be used in views.
Keywords: perf
Reporter | ||
Updated•11 years ago
|
Status: NEW → ASSIGNED
Summary: [tarako] Refactor app install, index code out of views/search.js → [tarako] Move indexing code out of views/search.js
Reporter | ||
Comment 2•11 years ago
|
||
Indexing code moved in https://github.com/mozilla/darjeeling/commit/caf4bd18836080d59586938787a9e215032de030
Summary: [tarako] Move indexing code out of views/search.js → [tarako] Move indexing code, app install code out of views/search.js
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → kngo
Reporter | ||
Comment 3•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•