794637 - /crash_trends remove the graph point for 0 data

Status: VERIFIED FIXED

(Socorro :: Webapp, task)

Description

[Matt Brandt [:mbrandt]]

Attachment: qa - bug

Is the message "0 total crashes for builds 0 Days old" redundant and can it be removed?

Steps to reproduce:
1. goto /crash_trends
2. Filter on a product with that has only a few crashes represented in the trends

Actual:
On days where there is little to no data, the hover points to view the meta-data on the graph overlap. -- see screenshot.

Suggested:
Either remove the data point for 0 entirely or when there is data (greater than 0 crashes for that day) remove the data point for zero crashes.

Comment 1

[[github robot]]

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

https://github.com/mozilla/socorro/commit/34173239bfced82dfec3589281869e25b98a1241
Only show tooltip on graph if report count is greater than 0 fixes bug 794637

https://github.com/mozilla/socorro/commit/bda2c40367d9ed0d6cc56803040c5f362a89a550
Merge pull request #846 from ossreleasefeed/crash-trends-remove-tooltip-for-zero-report-counts-794637

Only show tooltip on graph if report count is greater than 0 fixes bug 794637

Comment 2

[[github robot]]

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

https://github.com/mozilla/socorro/commit/84d6b38a7e5fb39f543f942205a50535aa6cfe4b
Only show tooltip on graph if report count is greater than 0 fixes bug 794637

https://github.com/mozilla/socorro/commit/f1a63bf3c13c7639045b99968cbd62bd3866531d
Merge pull request #847 from ossreleasefeed/crash-trends-remove-tooltip-for-zero-report-counts-794637

Only show tooltip on graph if report count is greater than 0 fixes bug 794637

Comment 3

[Schalk Neethling [:espressive]]

Sent and merged: https://github.com/mozilla/socorro/pull/846

TEST CASE
---------

Suggested === true;

Comment 4

[Matt Brandt [:mbrandt]]

QA verified on stage per comment 0 and comment 3. Thanks Schalk!