Skip to content

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
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 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
ewaostrowska added a commit to ewaostrowska/swagger-core that referenced this pull request Nov 6, 2025
Reproduces bug where ArraySchema.arraySchema.requiredMode is ignored.

Test 1 (testArraySchemaRequiredMode): Fails - demonstrates the bug
Test 2 (testWorkaround): Passes - validates workaround using schema.requiredMode

Related to swagger-api#4341, swagger-api#3438, swagger-api#4286
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants