Skip to content
This repository has been archived by the owner on May 26, 2020. It is now read-only.

Django 2.0 #433

Open
ronaldlangeveld opened this issue Apr 14, 2018 · 11 comments
Open

Django 2.0 #433

ronaldlangeveld opened this issue Apr 14, 2018 · 11 comments

Comments

@ronaldlangeveld
Copy link

Any plans for Django 2.0 support?

@kamilgregorczyk
Copy link

It seems to work for me, do you have any problems?

@joeyworld
Copy link

joeyworld commented Apr 16, 2018

If it is compatible with django 2.0, wouldn't be better if the docs could be updated? - #435 #436

@ronaldlangeveld
Copy link
Author

I got it working. :D Thanks guys!

@budiardy
Copy link

It doesn't work well with django 2.0.
The issue is: failed to overwrite 'JWT_RESPONSE_PAYLOAD_HANDLER' setting.
the default is:
'JWT_RESPONSE_PAYLOAD_HANDLER': 'rest_framework_jwt.utils.jwt_response_payload_handler',
want to become:
'JWT_RESPONSE_PAYLOAD_HANDLER': 'jwt_response_payload_handler',
i put the "def jwt_response_payload_handler" in the same page in the django settings file.
I guess it because the different version of Django

@jadhavmanoj
Copy link

Any update on support django 2.x? I am now evaluating which JWT auth package use for python 3.x & Django 2.x.

@krdeepak
Copy link

krdeepak commented Apr 26, 2019

Same here. I tried https://github.com/davesque/django-rest-framework-simplejwt, but this seems to be more widely adopted. Is someone working on django 2.0 official support?

@jadhavmanoj
Copy link

jadhavmanoj commented Apr 26, 2019

@krdeepak I used in Django 2.x. I don't see any issue. It works good.

@stunaz
Copy link

stunaz commented Apr 26, 2019

@jadhavmanoj but the tests are not against Django 2.x in this repo

@jadhavmanoj
Copy link

@stunaz true. currently, I am writing test cases for Django 2.1 support. I will send PR once I finished.

@jadhavmanoj
Copy link

@stunaz Please check PR #479 All test cases executed except for python version 3.3. tox has some issue for python version 3.3.

@stunaz
Copy link

stunaz commented Apr 27, 2019

cool @jadhavmanoj . Now, good luck to get it merged. this project is abandoned, hasnt get updated for years ... see #449

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

No branches or pull requests

7 participants