FastAPI-JSONAPI
FastAPI-JSONAPI copied to clipboard
fix for custom fields for includes (many-to-one direction)
by passing missing include_fields parameter.
fixes issue #101 for me.
@srepmub thank you for your PR. Please fix the formatting and order of the 'included' in the tests. After that, the changes can be merged into the main branch.
not entirely sure what you mean, but how about this..? (user1 and user2 are already in the right order though it seems?)
fix a formatting issue.
not sure why 'black' insists on putting certain things on a single line, but not similar code elsewhere. another attempt :P
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 91.44%. Comparing base (
a1a4fc5) to head (ad7bc3a).
Additional details and impacted files
@@ Coverage Diff @@
## main #104 +/- ##
=======================================
Coverage 91.44% 91.44%
=======================================
Files 44 44
Lines 2408 2408
Branches 279 279
=======================================
Hits 2202 2202
Misses 143 143
Partials 63 63
| Flag | Coverage Δ | |
|---|---|---|
| unittests | 91.44% <100.00%> (ø) |
Flags with carried forward coverage won't be shown. Click here to find out more.
| Files with missing lines | Coverage Δ | |
|---|---|---|
| fastapi_jsonapi/views/view_base.py | 93.05% <100.00%> (ø) |
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
okay now 'ruff' fails.. will try to fix
sorry for the delay. hope everything passes now.