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

Support HyperlinkedModelSerializer #73

Merged
merged 3 commits into from
Feb 9, 2015
Merged

Support HyperlinkedModelSerializer #73

merged 3 commits into from
Feb 9, 2015

Conversation

semente
Copy link
Contributor

@semente semente commented Feb 3, 2015

By providing the request object to the jwt_response_payload_handler we can use HyperlinkedModelSerializer for a "UserSerializer" and prevent the following errror:

AssertionError: `HyperlinkedIdentityField` requires the request in the serializer context. Add `context={'request': request}` when instantiating the serializer.

By providing the ``request`` object to the jwt_response_payload_handler we can use ``HyperlinkedModelSerializer`` for a "UserSerializer" and prevent the following errror:

{{{
AssertionError: `HyperlinkedIdentityField` requires the request in the serializer context. Add `context={'request': request}` when instantiating the serializer.
}}}
@jpadilla
Copy link
Owner

jpadilla commented Feb 4, 2015

@semente seems reasonable. Mind checking those failing tests?

@semente
Copy link
Contributor Author

semente commented Feb 8, 2015

You are cool now :)

jpadilla added a commit that referenced this pull request Feb 9, 2015
Support HyperlinkedModelSerializer
@jpadilla jpadilla merged commit 42efdf1 into jpadilla:master Feb 9, 2015
@jpadilla
Copy link
Owner

jpadilla commented Feb 9, 2015

@semente yeah, looks good now. Thanks!

@jpadilla jpadilla modified the milestone: 1.3.0 Release Mar 6, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants