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

Fixes #405 and #406 #407

Merged
merged 2 commits into from
Nov 17, 2016
Merged

Fixes #405 and #406 #407

merged 2 commits into from
Nov 17, 2016

Conversation

bastbnl
Copy link
Contributor

@bastbnl bastbnl commented Aug 9, 2016

Hi,

Thanks again for the project and the docs! I encountered two issues while running this code on #405 and #406. I've not tested the code on older versions of Django.

Should now work with old-style Middleware and the Django 1.10 new style
This should prevent others from running into #405 while trying to create both a swappable instance and swap it out at once
@niknokseyer
Copy link

Looks good @bastbnl

@poswald
Copy link
Collaborator

poswald commented Nov 17, 2016

Do you think it would be possible to extend the tox setup to include Django 1.10 here? I can merge this but I'd prefer to not do it blindly and think that would make sure that nothing in the future breaks it as well.

@synasius synasius self-assigned this Nov 17, 2016
@synasius synasius merged commit ebe09b7 into jazzband:master Nov 17, 2016
@synasius
Copy link
Contributor

@poswald we will update the CI to run the tests with django 1.10 and the development branch asap

@synasius
Copy link
Contributor

@poswald #425 added django 1.10 to the test matrix

@poswald
Copy link
Collaborator

poswald commented Nov 18, 2016

Excellent. Thank you for that.

@KeynesYouDigIt
Copy link

We have this abysmal practice of recreating and running migrations each time we deploy... is there any way to delay migrations or something to get around this?

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.

None yet

5 participants