Closed Bug 864581 Opened 13 years ago Closed 13 years ago

Add stats API

Categories

(Marketplace Graveyard :: API, defect, P1)

Product:
Marketplace Graveyard
Component:
API
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
2013-04-25

People

(Reporter: robhudson, Assigned: robhudson)

Details

(Whiteboard: p=2)

The code in mkt/stats is twisty and complicated and very difficult to work in. We also don't know what is used or isn't used. Now that we have monolith mkt/stats should simply be a view that loads templates to set up the javascript for charts which calls the stats API to get data. Moving away from highcharts also allows us to craft the API to accept its args more simply and return the data in a simpler form. WIP patch here before I realized I was working bugless: https://github.com/robhudson/zamboni/commit/0d4da2a New charts can use this new API. Old charts can migrate to it over time.
Priority: -- → P1
Target Milestone: --- → 2013-04-25
https://github.com/mozilla/zamboni/commit/3f71757 It's behind a waffle switch, "stats-api".
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
let me know if you want me to flip it
You need to log in before you can comment on or make changes to this bug.