Closed
Bug 788043
Opened 12 years ago
Closed 12 years ago
[socorro-crashstats] implement /daily aka "crashes per user"
Categories
(Socorro :: Webapp, task)
Socorro
Webapp
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rhelmer, Assigned: rhelmer)
References
()
Details
(Whiteboard: https://github.com/mozilla/socorro-crashstats/issues/2)
graphs are implemented, but not controls, and the table data is fake
Assignee | ||
Updated•12 years ago
|
Summary: implement /daily aka "crashes per user" → [socorro-crashstats] implement /daily aka "crashes per user"
Assignee | ||
Comment 1•12 years ago
|
||
There is some work done on this already: https://github.com/mozilla/socorro-crashstats/pull/58
Updated•12 years ago
|
Assignee: bsavage → sneethling
Updated•12 years ago
|
Target Milestone: --- → 21
Updated•12 years ago
|
Target Milestone: 21 → 22
Comment 2•12 years ago
|
||
I have two nits against the current PHP version of /daily that I think we should fix in the socorro-crashstats version right away: 1) Remove the "Total" row as it's pretty useless, esp. for Ratio. 2) Remove the % sign on the Ratio column - % would only be correct on a unit-less number, but this is in crashes/(100)ADU.
Updated•12 years ago
|
Target Milestone: 22 → 23
Assignee | ||
Updated•12 years ago
|
Assignee: sneethling → rhelmer
Comment 3•12 years ago
|
||
For Rob to start on this one, we first needs the following PR merged: https://github.com/mozilla/socorro-crashstats/pull/128 Then the work that has already been done for /daily lives here: https://github.com/ossreleasefeed/socorro-crashstats/tree/crashstats-implement-daily-788043
Assignee | ||
Comment 4•12 years ago
|
||
r? https://github.com/mozilla/socorro-crashstats/pull/145
Comment 5•12 years ago
|
||
Commits pushed to master at https://github.com/mozilla/socorro-crashstats https://github.com/mozilla/socorro-crashstats/commit/3f6286c6ba11f468d117ca1da9632cc53c5d6f94 implemented daily for crashstats fixes bug 788043 This builds on code from Schalk Neethling <sneethling@mozilla.com> and Brandon Savage <brandon@brandonsavage.net> https://github.com/mozilla/socorro-crashstats/commit/6198dcce20c4ab04c9086680e86347a5e6d09e11 Merge pull request #145 from rhelmer/bug788043-implement-daily implemented daily for crashstats fixes bug 788043
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 6•12 years ago
|
||
Commits pushed to master at https://github.com/mozilla/socorro-crashstats https://github.com/mozilla/socorro-crashstats/commit/fb55e2c2deaca2e645f25d7807696b3d44925f7c bug 788043 - fix unit tests https://github.com/mozilla/socorro-crashstats/commit/cda2dacfa532f3099dd3566d1c8dc49eda9bd897 Merge pull request #147 from rhelmer/bug788043-fix-busted-tests bug 788043 - fix unit tests
Comment 7•12 years ago
|
||
Commits pushed to master at https://github.com/mozilla/socorro-crashstats https://github.com/mozilla/socorro-crashstats/commit/ac7f46e4465432c595b544ba91a6d05f818af8cd bug 788043 - bugs found testing new daily implementation: * start with utcnow so mware response will yesterday * remove debugging from template (version number) * append percentage for ratio https://github.com/mozilla/socorro-crashstats/commit/e683f66a09de90a32d1375c1c6b0f417c764ea88 Merge pull request #154 from rhelmer/bug788043-regressions bug 788043 - bugs found testing new daily implementation:
Updated•12 years ago
|
Target Milestone: 23 → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•