Skip to content

Add support django 1.10 middleware #868

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

Closed
wants to merge 2 commits into from
Closed

Add support django 1.10 middleware #868

wants to merge 2 commits into from

Conversation

digitalashes
Copy link

Hook for support django 1.10 middleware

@codecov-io
Copy link

codecov-io commented Aug 12, 2016

Current coverage is 75.50% (diff: 78.94%)

Merging #868 into master will increase coverage by 0.05%

@@             master       #868   diff @@
==========================================
  Files            30         30          
  Lines          1654       1670    +16   
  Methods           0          0          
  Messages          0          0          
  Branches        246        247     +1   
==========================================
+ Hits           1248       1261    +13   
- Misses          327        329     +2   
- Partials         79         80     +1   

Powered by Codecov. Last update ffe1bd8...85aedf2

@matthiask
Copy link
Member

Thanks! The fix is mostly the same as in #864, but still with support for the automatic setup. If automatic setup were removed we should move forward with the simpler PR.

(I sent a message to django-users@, maybe chime in if you have an opinion regarding the automatic setup.)

@matthiask
Copy link
Member

I've merged #864 instead, therefore this PR is now obsolete.

Still, thanks for the contribution!

@matthiask matthiask closed this Aug 20, 2016
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