-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
update for django 1.9 #550
Conversation
The code looks good and I like it, but I have some concerns about backward compatibility, maybe we can keep the code in the |
I've implemented the mentioned change and merged the work. Thanks! |
Thanks dude. If you're ever in DC, first round is on me! |
I am trying to upgrade django 1.7 to 1.9.11 and after upgrade it, I am getting below error.
|
@sohamnavadiya did you install the package |
@omab I have installed social-auth-app-django (0.1.0)
|
@omab Now I am getting below exception stack.
|
@omab I solved above issue. By the way thanks for quick reply. I really appreciate your work. |
@sohamnavadiya no problem, do you mind posting the solution, I know that transitions aren't always smooths and could be useful to somebody else experiencing the same problem. |
Added some initialization code to a new ready method in the PythonSocialAuthConfig because of changes in Django. See https://docs.djangoproject.com/en/1.7/ref/applications/ and https://docs.djangoproject.com/en/1.7/releases/1.7/.
I'm a big fan and feel free to disregard if you already got some code in the works.
Thank you!