-
Notifications
You must be signed in to change notification settings - Fork 26
Merge attempt into django-oauth-toolkit #23
Comments
Excellent. Your help will be highly valuable to everyone :) |
I'll let you and your team know as soon as I can show progress. |
No worries, I think that having by now this solution that just works, nobody is in a hurry. Thank you. |
Since #24 is thankfully accepted, I'm going to start to work on this. |
Since #26 is merged, I think most of the work is done on updating this library. With your permission my next step would be the contact the jazzband, what have to be done in order to include most of this library in theirs. I keep you posted on any progress. |
Great job, thanks! |
@TM2500 Have you submitted a PR to django-oauth-toolkit? I don't think you need to join jazzband -- but of course it would be great if you did. Anyone can submit PRs. I suspect I know someone who would be interested in merging this ;-) |
@n2ygk I'm sorry to say, that my project, I depended on this library, evolved to a state, where Django was not the best fit anymore. So my intend to merge or get in touch with jazzband was and is no longer a high priority on my agenda. I propably should have closed that issue or mark it in another way as stale, Sorry for that -_- |
Hi there,
since there are a still some guys who wish that JWT be implemented in the django-oauth-toolkit and this repo is one of the few sane modules to successfully implemented it as a separate module I would like to make an attempt to merge it into the DOT-lib directly.
The first thing I would do, would be resolving issue #12 and make a PR into this repo and then I would refactor the module to integrate it nicely into DOT.
The text was updated successfully, but these errors were encountered: