Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Allow field extensions to return None from map_serializer_field #826

Merged

Conversation

awbacker
Copy link
Contributor

@awbacker awbacker commented Oct 2, 2022

A field can be marked as ignored when generating by using @extended_schema_field(None) or OpenApiTypes.NONE. This change extends the Extension processing so its behavior matches the decorator.

A field can be marked as ignored when generating by using
`@extended_schema_field(None)` or `OpenApiTypes.NONE`.  This change
extends the Extension processing so its behavior matches the decorator.
tfranzel added a commit that referenced this pull request Oct 5, 2022
@tfranzel tfranzel merged commit 5b63919 into tfranzel:master Oct 5, 2022
@tfranzel
Copy link
Owner

tfranzel commented Oct 5, 2022

thanks for the PR @awbacker. just solved a minor issue in the tests where you collided with another extension test. resolved it by giving your test it's own field test class.

@awbacker
Copy link
Contributor Author

awbacker commented Oct 5, 2022

Thanks you. Sorry I didn't notice that the PR was going directly here, thought I had time. And... i was just going to close that issue too, or comment.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants