-
-
Notifications
You must be signed in to change notification settings - Fork 158
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
OpenApi spec shows all fields for the models as nullable. #361
Comments
Hey Jules, |
Bit off-topic but: Do you really need typescript classes @JulesImmerzeel ? All these types are available as native browser interfaces already: On-topic: Would be nice to have required properties on the OpenAPI spec of course. |
Hey all, I am new to this package. I like it, but there is one inconvience I have noticed so far.
All fields are marked as nullable as you can see in this screenshot (this is just one example it's for all). One way to resolve this is by adding the RequiredAttribute to the properties that are not nullable.
The reason why it might be important to fix this:
Is there a reason why [Required] is not used or just something that was not taken into consideration?
If the later is the case I could have a look into it, as I am quiet familiar with WebAuthn specifications.
The text was updated successfully, but these errors were encountered: