graphs are implemented, but not controls, and the table data is fake
Summary: implement /daily aka "crashes per user" → [socorro-crashstats] implement /daily aka "crashes per user"
There is some work done on this already: https://github.com/mozilla/socorro-crashstats/pull/58
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.
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
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 <firstname.lastname@example.org> and Brandon Savage <email@example.com> https://github.com/mozilla/socorro-crashstats/commit/6198dcce20c4ab04c9086680e86347a5e6d09e11 Merge pull request #145 from rhelmer/bug788043-implement-daily implemented daily for crashstats fixes bug 788043
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
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
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:
You need to log in before you can comment on or make changes to this bug.