Closed
Bug 996665
Opened 11 years ago
Closed 11 years ago
[tarako] add collections to our API dumps
Categories
(Marketplace Graveyard :: API, defect, P1)
Tracking
(Not tracked)
RESOLVED
FIXED
2014-04-29
People
(Reporter: clouserw, Assigned: mstriemer)
References
Details
The dumps at http://firefox-marketplace-api.readthedocs.org/en/latest/topics/export.html are basically just tar files of the JSON for each app we have. It doesn't have collections at all. We should add them.
This likely means:
1) Add new API endpoints which will return collection details (priority 2: list/search collections)
2) Add documentation for the endpoints at http://firefox-marketplace-api.readthedocs.org/en/latest/index.html
3) Add any public collection (app_collections.is_public=1) to the bulk export cron that creates the tar files in the original link. I'd expect the same directory structure there, except use a /collections/ top level dir instead of /apps/
Comment 1•11 years ago
|
||
FWIW We already have collections endpoints - in /rocketfuel/ and in /feed/ (and well, in /fireplace/) - they are just not designed for public consumption.
A quick and dirty way to fix this would be to add a simple list of collection slugs to the app serializer (and es app serializer) and figure out whether we need extra endpoints later.
Updated•11 years ago
|
Assignee: nobody → mstriemer
Target Milestone: --- → 2014-04-22
Updated•11 years ago
|
Priority: -- → P1
Summary: add collections to our API dumps → [tarako] add collections to our API dumps
Assignee | ||
Comment 2•11 years ago
|
||
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•11 years ago
|
||
https://github.com/mozilla/zamboni/commit/2758c9ecdab853512fb9ff0d494273502e8ec667
You can now get export with collections (from dev) at https://marketplace-dev-cdn.allizom.org/dumped-apps/tarballs/2014-04-22.tgz.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•11 years ago
|
Target Milestone: 2014-04-22 → 2014-04-29
Assignee | ||
Comment 4•11 years ago
|
||
This is currently dumping all app data in the export, I created bug 999598 to reduce the duplication.
Comment 5•11 years ago
|
||
Can you please add some STRs to this bug or mark it as [qa-] ?
You need to log in
before you can comment on or make changes to this bug.
Description
•