Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[registration] Provide serializer context to TokenSerializer/JWTSerializer #478

Closed
nemesifier opened this issue Nov 8, 2018 · 1 comment
Closed

[registration] Provide serializer context to TokenSerializer/JWTSerializer #478

nemesifier opened this issue Nov 8, 2018 · 1 comment

Comments

@nemesifier
Copy link

If the serializer context is not passed to TokenSerializer/JWTSerializer, the ways in which those serializers can be extended is greatly impaired. I'm going to send a patch to fix this. It's a very small addition which I think makes sens to do in the core code of django-rest-auth.
nemesifier added a commit to nemesifier/django-rest-auth that referenced this issue Nov 8, 2018
@nemesifier
Pass serializer context in RegisterView.get_response_data Tivix#478
19b3f5d 
Pass serializer_context to TokenSerializer/JWTSerializer
so extensions of these serializers can make use of it
to implement customised logic.

Closes Tivix#478
nemesifier added a commit to nemesifier/django-rest-auth that referenced this issue Nov 8, 2018
@nemesifier
Pass serializer context in RegisterView.get_response_data Tivix#478
870f414 
Pass serializer_context to TokenSerializer/JWTSerializer
so extensions of these serializers can make use of it
to implement custom logic.

Closes Tivix#478
nemesifier added a commit to nemesifier/django-rest-auth that referenced this issue Nov 8, 2018
@nemesifier
Pass serializer context in RegisterView.get_response_data Tivix#478
cf9f469 
Pass serializer context to TokenSerializer/JWTSerializer
so extensions of these serializers can make use of it
to implement custom logic.

Closes Tivix#478
@nemesifier nemesifier mentioned this issue Nov 8, 2018
Closed
@githubuser88442 githubuser88442 mentioned this issue Apr 21, 2020
Closed
@nemesifier
Copy link
Author

Resolved in iMerica/dj-rest-auth#52.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Pass serializer context in RegisterView.get_response_data #478 nemesifier/django-rest-auth
1 participant
@nemesifier