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

Use validated_data to access validated data #401

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xrmx
Copy link

@xrmx xrmx commented Dec 6, 2017

Instead of object which is not documented in DRF and for some
reason does not work here.

Instead of object which is not documented in DRF and for some
reason does not work here.
@codecov
Copy link

codecov bot commented Dec 6, 2017

Codecov Report

Merging #401 into master will decrease coverage by 0.45%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #401      +/-   ##
==========================================
- Coverage   90.67%   90.22%   -0.46%     
==========================================
  Files          14       12       -2     
  Lines         847      818      -29     
  Branches       29       29              
==========================================
- Hits          768      738      -30     
- Misses         66       67       +1     
  Partials       13       13
Flag Coverage Δ
#codecov 90.22% <100%> (-0.46%) ⬇️
#dj110 86.91% <100%> (-0.57%) ⬇️
#dj111 86.91% <100%> (-0.57%) ⬇️
#dj18 89.36% <100%> (-0.49%) ⬇️
#dj19 89.36% <100%> (-0.49%) ⬇️
#drf31 89.36% <100%> (-0.49%) ⬇️
#drf32 89.36% <100%> (-0.49%) ⬇️
#drf33 89.36% <100%> (-0.49%) ⬇️
#drf34 90.22% <100%> (-0.46%) ⬇️
#drf35 89.85% <100%> (-0.47%) ⬇️
#drf36 89.85% <100%> (-0.47%) ⬇️
#py27 90.22% <100%> (-0.46%) ⬇️
#py33 88.99% <100%> (-0.5%) ⬇️
#py34 89.85% <100%> (+0.36%) ⬆️
#py35 86.91% <100%> (?)
#py36 86.91% <100%> (?)
Impacted Files Coverage Δ
rest_framework_jwt/views.py 92.5% <100%> (ø) ⬆️
tests/test_serializers.py 95.31% <100%> (ø) ⬆️
rest_framework_jwt/compat.py 69.56% <0%> (-4.35%) ⬇️
rest_framework_jwt/models.py
rest_framework_jwt/utils.py

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0a0bd40...09081ec. Read the comment docs.

@codecov
Copy link

codecov bot commented Dec 6, 2017

Codecov Report

Merging #401 into master will decrease coverage by 0.45%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #401      +/-   ##
==========================================
- Coverage   90.67%   90.22%   -0.46%     
==========================================
  Files          14       12       -2     
  Lines         847      818      -29     
  Branches       29       29              
==========================================
- Hits          768      738      -30     
- Misses         66       67       +1     
  Partials       13       13
Flag Coverage Δ
#codecov 90.22% <100%> (-0.46%) ⬇️
#dj110 86.91% <100%> (-0.57%) ⬇️
#dj111 86.91% <100%> (-0.57%) ⬇️
#dj18 89.36% <100%> (-0.49%) ⬇️
#dj19 89.36% <100%> (-0.49%) ⬇️
#drf31 89.36% <100%> (-0.49%) ⬇️
#drf32 89.36% <100%> (-0.49%) ⬇️
#drf33 89.36% <100%> (-0.49%) ⬇️
#drf34 90.22% <100%> (-0.46%) ⬇️
#drf35 89.85% <100%> (-0.47%) ⬇️
#drf36 89.85% <100%> (-0.47%) ⬇️
#py27 90.22% <100%> (-0.46%) ⬇️
#py33 88.99% <100%> (-0.5%) ⬇️
#py34 89.85% <100%> (+0.36%) ⬆️
#py35 86.91% <100%> (?)
#py36 86.91% <100%> (?)
Impacted Files Coverage Δ
tests/test_serializers.py 95.31% <100%> (ø) ⬆️
rest_framework_jwt/views.py 92.5% <100%> (ø) ⬆️
rest_framework_jwt/compat.py 69.56% <0%> (-4.35%) ⬇️
rest_framework_jwt/models.py
rest_framework_jwt/utils.py

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0a0bd40...09081ec. Read the comment docs.

@xrmx
Copy link
Author

xrmx commented Apr 6, 2018

Friendly ping :)

@xrmx
Copy link
Author

xrmx commented Aug 17, 2018

@blueyed sorry to bother, any chance you can merge this please?

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

Successfully merging this pull request may close these issues.

1 participant