986634 - Remove unused icons from Fireplace-specific API endpoints

Status: VERIFIED FIXED

(Marketplace Graveyard :: API, defect, P4)

Description

[Christopher Van Wiemeersch [:cvan]]

https://marketplace.firefox.com/api/v1/fireplace/search/featured/?region=us&lang=en-US

We use only the 64px-sized icons. Let's remove the URLs for the 16px-, 48px-, and 128px-sized icons.

Comment 1

[Mathieu Pillard [:mat]]

Fixed in https://github.com/mozilla/zamboni/commit/fa0a6b6bab08e9d9dbfebf7e3a1e013a62298d14

STR:
- Load consumer pages homepage with developer tools network tab opened, make sure icons still show up
- Look in the XHR section, for the call to /api/v1/fireplace/search/featured/
- Make sure each "icons" object in the results only has one entry (64, which is the icon size we want)
- Repeat with a detail page, this time looking for the call to /api/v1/fireplace/app/<yourapp> (reload the page first)

Comment 2

[Iulian Timis]

Verified as fixed in https://marketplace.allizom.org/ on FF31 (Win 7).
Postfix screencast http://screencast.com/t/enQ4ZgmcZac9
Closing bug.