Add devices to Signature Summary

VERIFIED FIXED in 59

Status

Socorro
Webapp
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: brandon, Assigned: brandon)

Tracking

unspecified

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)

Comment 1

4 years ago
Commits pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/4d843652fb28269a53083d5b8f91828781bad73c
Fixes Bug 914222 - Adds an Android device summary to Signature Summary.

https://github.com/mozilla/socorro/commit/e6c380516ea385e04522c72261a004e32f62de54
Merge pull request #1481 from brandonsavage/bug914222devices

Fixes Bug 914222 - Adds an Android device summary to Signature Summary.

Updated

4 years ago
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
(Assignee)

Comment 2

4 years ago
Steps to test: verify that the signature summary now contains an Android devices report. Use FennecAndroid for best results.
Target Milestone: --- → 59

Comment 3

4 years ago
Commit pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/c3315f6cf5c58645216dafdab7866630484d439c
Merge pull request #1488 from brandonsavage/bug914222devices

Fixes Bug 914816 - Changes to the GROUP BY rules for device summary, and reordering the columns per KaiRo's request.
Created attachment 803043 [details]
qa_reopened.png

Reopened:

zero's in decimal places should be removed.

Updated

4 years ago
Status: RESOLVED → REOPENED
OS: Mac OS X → All
Hardware: x86 → All
Resolution: FIXED → ---
(Assignee)

Comment 5

4 years ago
I have to disagree; we retain the significant digits for other reports, most often 100.00%. It would create readability challenges to remove the significant digits, even when they're zero.
Status: REOPENED → RESOLVED
Last Resolved: 4 years ago4 years ago
Resolution: --- → FIXED
Fair enough .. it's just my nit (I can swallow it) :)

Bumping to QA verified, nicely done sir
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.