[socorro-crashstats] implement /daily aka "crashes per user"

RESOLVED FIXED

Status

Socorro
Webapp
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: rhelmer, Assigned: rhelmer)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: https://github.com/mozilla/socorro-crashstats/issues/2, URL)

(Assignee)

Description

6 years ago
graphs are implemented, but not controls, and the table data is fake
(Assignee)

Updated

6 years ago
Blocks: 788003
(Assignee)

Updated

6 years ago
Summary: implement /daily aka "crashes per user" → [socorro-crashstats] implement /daily aka "crashes per user"
(Assignee)

Comment 1

6 years ago
There is some work done on this already:
https://github.com/mozilla/socorro-crashstats/pull/58
Blocks: 788055
Assignee: bsavage → sneethling
Target Milestone: --- → 21
Target Milestone: 21 → 22

Comment 2

6 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.
Target Milestone: 22 → 23

Updated

6 years ago
Depends on: 794457
(Assignee)

Updated

6 years ago
Assignee: sneethling → rhelmer
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

Comment 5

6 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

6 years ago
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED

Comment 7

6 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

6 years ago
Target Milestone: 23 → ---
You need to log in before you can comment on or make changes to this bug.