-
Notifications
You must be signed in to change notification settings - Fork 299
Allow parser context to be None #981
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
Conversation
Just as a follow up I discovered this issue while implementing PR #898. So it is basically an issue which can occur during testing.
Codecov Report
@@ Coverage Diff @@ ## master #981 +/- ## ======================================= Coverage 96.82% 96.82% ======================================= Files 65 65 Lines 3904 3906 +2 ======================================= + Hits 3780 3782 +2 Misses 124 124
Continue to review full report at Codecov.
|
Default parameter of `parser_context` was already set to `None` but would had always raised an error. DRF parser allows `parser_context` to be `None` to so brings DJA to use the same API.
9bdbb34 to
fcbca4f
Compare
Description of the Change
Default parameter of
parser_contextwas already set toNonebut would had always raised an error when it was actuallyNone.DRF parser allows
parser_contextto beNoneso this PR makes the DJA parser fully compatible with DRF API.Checklist
CHANGELOG.mdupdated (only for user relevant changes)AUTHORS