Skip to content
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

Adds configuration settings to influence required annotation #1563

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tsinghammer
Copy link

Sometimes it would be useful, also with an OpenApi3 schema to generate non-required fields as Required.Default.
Do you think it makes sense to follow such a toggle and elaborate it more?

@jkone27
Copy link

jkone27 commented Jan 18, 2024

is this available for merge @RicoSuter ? this seettings seems useful instead of having to register a custom serializer to allow nullable properties (openapi schemas can often be unprecise in the real world)

@mjmadhu
Copy link

mjmadhu commented Apr 23, 2024

Hitting RicoSuter/NSwag#3851 issue.. can anyone help with ETA for fix??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants