Skip to content

feat: replace required attribute on @Schema by requiredMode #4286

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Nov 2, 2022
Merged

feat: replace required attribute on @Schema by requiredMode #4286

merged 2 commits into from
Nov 2, 2022

Conversation

@frantuma
Copy link
Contributor

@frantuma frantuma commented Nov 2, 2022

Includes and replaces #4221

nvervelle and others added 2 commits November 2, 2022 11:37
@nvervelle @frantuma
feat: replace required attribute on @Schema by requiredMode
762f76a 
Add a property requiredMode on @Schema annotations that allows deciding if the heuristics on annotations like @NotNull should apply, so we can have a property annotated @NotNull but still have required = false in the openapi spec
@frantuma
fix: fix javadocs for Schema.requiredMode
f0bc3ff
@frantuma frantuma merged commit 79339b3 into master Nov 2, 2022
@frantuma frantuma deleted the ft/4213-Replace-required-by-requiredMode branch November 2, 2022 10:47
This was referenced Nov 2, 2022
Closed
Closed
Closed
@leedsalex leedsalex mentioned this pull request Dec 13, 2022
Closed
@adase11 adase11 mentioned this pull request Dec 30, 2022
Closed
@keigo-brook keigo-brook mentioned this pull request Dec 31, 2022
Open
@swaechter swaechter mentioned this pull request Mar 22, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@frantuma @nvervelle