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

Fixing Depreciation, Saving and Performance Tweaks #104

Merged
merged 5 commits into from
Mar 13, 2016

Conversation

Wrhector
Copy link
Contributor

I can break this up, but simply this fixes:

  • Django deprecation warnings.
  • A previous commit broke sql and profile saving entirely.
  • Stop attaching the profiler every run if not used.
  • Better update of total queries (Though this really could be refactored into one update)

avelis added a commit that referenced this pull request Mar 13, 2016
Fixing Depreciation, Saving and Performance Tweaks
@avelis avelis merged commit 5a82689 into jazzband:master Mar 13, 2016
@avelis
Copy link
Collaborator

avelis commented Mar 13, 2016

@Wrhector Thanks for fixing the broken collection. Truly was a mistake on my part. Also, I appreciate the effort to the project.

In the interest of progress I have merged it in!

@wm3ndez
Copy link
Member

wm3ndez commented Mar 30, 2016

@Wrhector 958d843 breaks django=1.7.x, is this version of django no longer supported?

@Wrhector
Copy link
Contributor Author

Django 1.7 has been end of life for 6 months now, though oddly travis passed checks with Django 1.7, I'll write a patch to support both.

@wm3ndez
Copy link
Member

wm3ndez commented Mar 30, 2016

@Wrhector I already sent a patch, thanks

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

Successfully merging this pull request may close these issues.

3 participants