-
Notifications
You must be signed in to change notification settings - Fork 301
Render meta fields of included resource #883
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.
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
Render meta fields of included resource #883
Conversation
Codecov Report
@@ Coverage Diff @@
## master #883 +/- ##
=======================================
Coverage 97.80% 97.80%
=======================================
Files 60 60
Lines 3321 3325 +4
=======================================
+ Hits 3248 3252 +4
Misses 73 73
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot for this PR. This brings certainly more consistency to DJA.
I have added some inline comments. Please also add a changelog entry. Thanks.
75c4d42
to
3a3032a
Compare
Thanks for your contributions. As mentioned above I adjusted your PR a bit but all the rest looked good and ready for merging. |
Thanks, and sorry I could not complete it myself. Glad i could help |
Description of the Change
When adding serializer as a field the meta fields are not rendered, This change provides the rendering capability, test and some fixes of related tests.
Checklist
CHANGELOG.md
updated (only for user relevant changes)AUTHORS