Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add time ranges to quickly adjust the main view of the crash website #29

Closed
foss- opened this issue Dec 10, 2016 · 7 comments
Closed

add time ranges to quickly adjust the main view of the crash website #29

foss- opened this issue Dec 10, 2016 · 7 comments

Comments

@foss-
Copy link

foss- commented Dec 10, 2016

Add adjustable time ranges for the main view. Suggestion:

  • 3 days
  • 7 days (default)
  • 14 days
  • 30 days (one month)

To get an idea, attached you find how mozilla solved this for firefox crash stats.
ff crash stats

This should come useful, since it allows to get a quick overview of how versions and crashes evolve.

@mmohrhard
Copy link
Owner

This can be easily added as we already have something similar.

https://github.com/mmohrhard/crash/blob/master/django/crashreport/stats/templates/stats/version.html#L20 already contains something like this for the version view.

So this requires to adapt the https://github.com/mmohrhard/crash/blob/master/django/crashreport/stats/templates/stats/main.html file with the corresponding template code and teach https://github.com/mmohrhard/crash/blob/master/django/crashreport/stats/views.py#L91 about the additional get parameter. Don't forget to add the parameter to the corresponding URLs.

@amitdaga
Copy link

@mmohrhard Have you assigned anyone for this task? I can take this up if this is still unassigned.

@foss-
Copy link
Author

foss- commented Apr 17, 2017

It pretty sure is unassigned

@mmohrhard
Copy link
Owner

Yes it is still unassigned.

@amitdaga
Copy link

#39 Setup would need some time. Will verify these changes after that.

@mmohrhard
Copy link
Owner

Thanks a lot for the patch. It has been merged and deployed to production.

For the setup current master has already some helpful code and can populate the DB with initial data through ./manage.py setup including a bit of data for the charts.

Are you interested in another simple task?

@amitdaga
Copy link

@mmohrhard Yes. Can have a look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants