912132 - Ability to configure columns shown on report/list page

Status: VERIFIED FIXED

(Socorro :: Webapp, task)

Description

[Peter Bengtsson [:peterbe]]

Attachment: Screen Shot 2013-09-03 at 10.53.21 AM.png

The columns on the Reports tab on, for example, http://socorro/report/list?product=Firefox&range_value=7&range_unit=days&date=2013-09-01&signature=nsContentUtils%3A%3AGetSubjectPrincipal%28%29&version=Firefox%3A26.0a1 to be something you can, with a cookie, override and change.

Comment 1

[Peter Bengtsson [:peterbe]]

Selena,

My objective is to extend the reports page (Reports tab on the reports list) with columns from the raw crash. http://cl.ly/RSUn

Currently that table is generated by querying socorro.external.postgresql.report:Report.get_list() which, it appears, simply does a plain select from `reports`.

Can I just add an inner join on `raw_crash` and join on uuid and then include `raw_crash` which is a JSON type?

Does that sound right? Then I can unpack that massive JSON thing when I render the template based on which columns specifically the user has chosen to see.

Comment 2

[Peter Bengtsson [:peterbe]]

Crap crap crap!
I just made that comment on the wrong bug. 

It was supposed to be:
https://bugzilla.mozilla.org/show_bug.cgi?id=889041#c2

Comment 3

[[github robot]]

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

https://github.com/mozilla/socorro/commit/a1950641afa2b3e4fc172e01caa1096b31da8455
fixes bug 912132 - Ability to configure columns shown on report/list page, r=adriangaudebert

Comment 4

[Peter Bengtsson [:peterbe]]

This is a new feature but I'll use familiar language...

Steps to reproduce:

1. Go to Top Crasher
2. Click on any signature
3. Click on the Reports tab
4. Take note of what columns are loaded by default it should be [0]
5. Click the "Change columns in table" link above the table of reports. This should take you to the bottom of the page
6. Add some columns. Remove some columns. Click "Save and reload" (take note of which you changed)

Expected outcome:
1. The table columns should reflect what you changed in step 6. 
2. Reloading the page without changing any columns should remember the columns from step 6. 


[0] 
* Date
* Dup
* Product
* Version
* Build
* OS
* Build Arch
* Reason
* Address
* Crash Type
* Uptime
* Install Time
* Comments

Comment 5

[Lonnen :lonnen]

Steps to reproduce worked on stage. Tried reloading the page, loading a new report, and loading a new report from a different product/version combo and my column choices were retained across all of them.