django-rest-framework-jwt icon indicating copy to clipboard operation
django-rest-framework-jwt copied to clipboard

Use validated_data to access validated data

Open xrmx opened this issue 6 years ago • 4 comments

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

xrmx avatar Dec 06 '17 10:12 xrmx

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%) :arrow_down:
#dj110 86.91% <100%> (-0.57%) :arrow_down:
#dj111 86.91% <100%> (-0.57%) :arrow_down:
#dj18 89.36% <100%> (-0.49%) :arrow_down:
#dj19 89.36% <100%> (-0.49%) :arrow_down:
#drf31 89.36% <100%> (-0.49%) :arrow_down:
#drf32 89.36% <100%> (-0.49%) :arrow_down:
#drf33 89.36% <100%> (-0.49%) :arrow_down:
#drf34 90.22% <100%> (-0.46%) :arrow_down:
#drf35 89.85% <100%> (-0.47%) :arrow_down:
#drf36 89.85% <100%> (-0.47%) :arrow_down:
#py27 90.22% <100%> (-0.46%) :arrow_down:
#py33 88.99% <100%> (-0.5%) :arrow_down:
#py34 89.85% <100%> (+0.36%) :arrow_up:
#py35 86.91% <100%> (?)
#py36 86.91% <100%> (?)
Impacted Files Coverage Δ
rest_framework_jwt/views.py 92.5% <100%> (ø) :arrow_up:
tests/test_serializers.py 95.31% <100%> (ø) :arrow_up:
rest_framework_jwt/compat.py 69.56% <0%> (-4.35%) :arrow_down:
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[bot] avatar Dec 06 '17 11:12 codecov[bot]

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%) :arrow_down:
#dj110 86.91% <100%> (-0.57%) :arrow_down:
#dj111 86.91% <100%> (-0.57%) :arrow_down:
#dj18 89.36% <100%> (-0.49%) :arrow_down:
#dj19 89.36% <100%> (-0.49%) :arrow_down:
#drf31 89.36% <100%> (-0.49%) :arrow_down:
#drf32 89.36% <100%> (-0.49%) :arrow_down:
#drf33 89.36% <100%> (-0.49%) :arrow_down:
#drf34 90.22% <100%> (-0.46%) :arrow_down:
#drf35 89.85% <100%> (-0.47%) :arrow_down:
#drf36 89.85% <100%> (-0.47%) :arrow_down:
#py27 90.22% <100%> (-0.46%) :arrow_down:
#py33 88.99% <100%> (-0.5%) :arrow_down:
#py34 89.85% <100%> (+0.36%) :arrow_up:
#py35 86.91% <100%> (?)
#py36 86.91% <100%> (?)
Impacted Files Coverage Δ
tests/test_serializers.py 95.31% <100%> (ø) :arrow_up:
rest_framework_jwt/views.py 92.5% <100%> (ø) :arrow_up:
rest_framework_jwt/compat.py 69.56% <0%> (-4.35%) :arrow_down:
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[bot] avatar Dec 06 '17 11:12 codecov[bot]

Friendly ping :)

xrmx avatar Apr 06 '18 09:04 xrmx

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

xrmx avatar Aug 17 '18 13:08 xrmx