search

bmihelac / ra-data-django-rest-framework

react-admin data provider for Django REST framework
MIT License
100 stars 28 forks source link

Add JWT token auth provider based on djangorestframework-simplejwt #36

Closed barseghyanartur closed 3 years ago

barseghyanartur commented 3 years ago

As proposed/discussed in #34

bmihelac commented 3 years ago

@barseghyanartur thanks for this pull request. Code looks good but I do not have an easy way to test. Do you think it would be easiest to add to jwt to example app?

barseghyanartur commented 3 years ago

@barseghyanartur thanks for this pull request. Code looks good but I do not have an easy way to test. Do you think it would be easiest to add to jwt to example app?

Great! Thanks for checking. I thought tests should be sufficient, but sure - I will add a working example (later today/tomorrow) and let you know.

barseghyanartur commented 3 years ago

@bmihelac:

Example app is updated. Please, check.

bmihelac commented 3 years ago

@barseghyanartur merged, thanks!

barseghyanartur commented 3 years ago

@bmihelac:

Thanks! Could a npmjs release be expected one day soon? :)