907855 - Cache bugzilla summaries in localStorage for Top Crasher to load faster

Status: VERIFIED FIXED

(Socorro :: Webapp, task)

Description

[Peter Bengtsson [:peterbe]]

Moar caching -> fewer GET requests.

Comment 1

[Peter Bengtsson [:peterbe]]

There might be a better way. 
Perhaps we can instead add include_fields=status,summary and pull that down server-side. Then we can better cache things and won't need any XHR. Maybe.

Comment 2

[Peter Bengtsson [:peterbe]]

Pull request: https://github.com/mozilla/socorro/pull/1503

Comment 3

[Peter Bengtsson [:peterbe]]

let's not proceed with this till https://bugzilla.mozilla.org/show_bug.cgi?id=915430 is landed. 
But I sure would appreciate a review :)

Comment 4

[[github robot]]

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

https://github.com/mozilla/socorro/commit/f8596f0099202cbd3b037cc76cfc06de6cb38b9a
fixes bug 907855 - cache bugzilla summaries for fasster Top Crasher, r=rhelmer

Comment 5

[Peter Bengtsson [:peterbe]]

I can't believe we missed this. The `Bugzilla` namespace is taken by report_list_bugzilla.js
I think I might have missed that since I worked in two separate branches off master and it was not noticed until after I rebased and merged.

Comment 6

[Peter Bengtsson [:peterbe]]

New PR: https://github.com/mozilla/socorro/pull/1525
:rhelmer r?

Comment 7

[Robert Helmer [:rhelmer]]

(In reply to Peter Bengtsson [:peterbe] from comment #5)
> I can't believe we missed this. The `Bugzilla` namespace is taken by
> report_list_bugzilla.js
> I think I might have missed that since I worked in two separate branches off
> master and it was not noticed until after I rebased and merged.

oops :)

Comment 8

[Peter Bengtsson [:peterbe]]

Steps to reproduce:

1. Load Top Crasher
2. All the bug links should have a more advanced tooltip that "Find out more information in Bugzilla"
3. Hovering over any of the links should show a little pop-up thing
4. Links to bugs that have been "closed" should have a strike through style
5. Reloading the page should not break that.
6. Click any signature (that has bugs), click the Bugzilla tab
7. Expect bugs to work there equally too

Comment 9

[[github robot]]

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

https://github.com/mozilla/socorro/commit/ed12ca2237eb2e81cbb2331b964e4c47c5cb7613
fixes bug 907855 - Bugzilla namespace taken, r=lonnen

Comment 10

[Matt Brandt [:mbrandt]]

QA verified on stage - bugzilla tool tips on /topcrasher/ work (including the bugzilla urls):

https://crash-stats.allizom.org/topcrasher/products/Firefox/versions/24.0b?days=7