805314 - [socorro-crashstats] home page graph never changes its plot values based on number of days

Status: VERIFIED FIXED

(Socorro :: Webapp, task)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Screenshot of Django-rewrite

STR:

1. Load http://crash-stats-new-dev.allizom.org/home/products/Firefox
2. Note the graph dates (October 18th - October 23rd)
3. Switch back and forth between 3, 7, 14 days (try http://crash-stats-new-dev.allizom.org/home/products/Firefox#duration=14)

Actual:

Graph dates always remain fixed

Comment 1

[Stephen Donner [:stephend] Not actively reading bugmail]

(BTW, same for By Build Date: http://crash-stats-new-dev.allizom.org/home/products/Firefox#duration=14:date_range_type=build)

Comment 2

[Robert Helmer [:rhelmer]]

Schalk, could you take a look please? The behavior of this has changed quite a bit since the mobeta landing (uses AJAX now, I notice it uses #duration and ?days, etc), not sure what the "right" way to fix this is :)

Comment 3

[Schalk Neethling [:espressive]]

Super weird, I am sure this all worked just fine but, nevertheless, taking a look.

Comment 4

[Schalk Neethling [:espressive]]

https://github.com/mozilla/socorro-crashstats/pull/224 r?

Comment 5

[[github robot]]

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

https://github.com/mozilla/socorro-crashstats/commit/6274b553ac5fa371124f17de0233b39b74848cd2
pass correct param names from view to model fixes bug 805314

https://github.com/mozilla/socorro-crashstats/commit/9e6282b19e365698364116dec8855c1fa89fe875
Merge pull request #224 from ossreleasefeed/bug805314-plot-values-dont-change-for-days

pass correct param names from view to model fixes bug 805314

Comment 6

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED; thanks!