Skip to content

Conversation

@lcary
Copy link
Contributor

@lcary lcary commented May 21, 2018

This change creates a ReadOnlyModelViewSet with the required prefetch mixins.

Description of the Change

Checklist

  • PR only contains one change (considered splitting up PR)
  • unit-test added
  • documentation updated
  • changelog entry added to CHANGELOG.md
  • author name in AUTHORS

This change creates a ReadOnlyModelViewSet with the required prefetch mixins.
@lcary
Copy link
Contributor Author

lcary commented May 21, 2018

I'll tick off the rest of the checklist in time.

@codecov-io
Copy link

codecov-io commented May 21, 2018

Codecov Report

Merging #439 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #439      +/-   ##
==========================================
+ Coverage   91.92%   91.92%   +<.01%     
==========================================
  Files          57       57              
  Lines        2984     2986       +2     
==========================================
+ Hits         2743     2745       +2     
  Misses        241      241
Impacted Files Coverage Δ
rest_framework_json_api/views.py 95.12% <100%> (+0.06%) ⬆️

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 409fb65...9246279. Read the comment docs.

@sliverc
Copy link
Member

sliverc commented May 22, 2018

I thought update adding this in the past as well but never really got around. Hope you will get around the other points in the checklist and then I am happy to merge it.

@sliverc sliverc self-requested a review May 22, 2018 15:01
Copy link
Member

@sliverc sliverc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! Thanks for your contribution.

@sliverc sliverc merged commit bea4743 into django-json-api:master May 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants