751166 - [stage] New crash trends report needs a <title>

Status: VERIFIED FIXED

(Socorro :: Webapp, task)

Description

[Laura Thomson :laura]

The title on https://crash-stats.allizom.org/crash_trends is currently "New Report for..."

Can we land and cherry pick for today's release?

Comment 1

[[github robot]]

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

https://github.com/mozilla/socorro/commit/9708a21199fabb3a260d503fab02e49955ff7a73
adding proper title fixes bug 751166

https://github.com/mozilla/socorro/commit/8449e9c412984fa9c9db641d637a7632b422ed68
Merge pull request #545 from ossreleasefeed/crash-trends-needs-title-751166

adding proper title fixes bug 751166

Comment 2

[Matt Brandt [:mbrandt]]

On dev it appears that the title is not dynamically updating - it perpetually reads with the product/version that was originally loaded - in this case "Crash Trends Report For Firefox 15.0a1."

Comment 3

[Matt Brandt [:mbrandt]]

Bumping back to resolved fixed - per a conversation on irc with espressive I'll file a new bug for comment 2. We shouldn't block the release on this.

Comment 4

[Matt Brandt [:mbrandt]]

I am full of fail today - I commented in the wrong bug.

Comment 5

[Matt Brandt [:mbrandt]]

Bumping to verified fixed on dev (ignore comment 3 and comment 4) - the page title dynamically updates with the product and version number of what the report is being generated on.

Comment 6

[[github robot]]

Commit pushed to stage at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/dd3ad81a97c450af714cb8d654e18efa10b0cf6c
adding proper title fixes bug 751166

Comment 7

[Matt Brandt [:mbrandt]]

Attachment: qa - verified

QA verified on dev and stage - nice work espressive. the new report looks great.

Comment 8

[[github robot]]

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

https://github.com/mozilla/socorro/commit/9708a21199fabb3a260d503fab02e49955ff7a73
adding proper title fixes bug 751166

https://github.com/mozilla/socorro/commit/8449e9c412984fa9c9db641d637a7632b422ed68
Merge pull request #545 from ossreleasefeed/crash-trends-needs-title-751166