-
Notifications
You must be signed in to change notification settings - Fork 107
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 dependencies #147
base: master
Are you sure you want to change the base?
Update dependencies #147
Conversation
d7e08e7
to
79b874d
Compare
79b874d
to
7ae7eae
Compare
I would love to have this one solved, but I would like to hear @PedroBern 's opinion on it first. @PabloCastellano how can I test this best? |
@pors Try it:
|
17d38f2
to
7ae7eae
Compare
I can't get it to work for Django 4 (which is the only reason for me to upgrade django-graphql-jwt>=0.3.2). It does work for 0.3.1, but that doesn't work with Django 4. Sorry, but I think it is best to abandon this project and find an alternative. |
for me this branch works with
|
I still think this PR will break things for people. See #23. It doesn't say there what the actual incompatibility is though. Can someone help out there? @joshuachinemezu @PedroBern @ulgens @panosangelopoulos ? |
Supersedes #121, #116, #45
Fixes #111, #133 and #146