Skip to content

Conversation

arnedesmedt
Copy link
Contributor

@arnedesmedt arnedesmedt commented Nov 24, 2023

Q A
Branch? 3.2
License MIT

The generated open api spec is currently invalid for open api 3.1.
exclusiveMinimum and exclusiveMaximum are still used as boolean, while properties minimum and exclusiveMinimum are merged into one property exclusiveMinimum.

See https://www.openapis.org/blog/2021/02/16/migrating-from-openapi-3-0-to-3-1-0

The only problem I see now, is that users can configure their open api version.
But I don't know how I can toggle the logic between 3.0 and 3.1.
Any help is appreciated. Thx!

@arnedesmedt arnedesmedt changed the base branch from main to 3.2 November 24, 2023 09:51
…n api

minimum and exclusiveMinimum are merged into one property. The same goes
for maximum.

See
https://www.openapis.org/blog/2021/02/16/migrating-from-openapi-3-0-to-3-1-0
for more details.
@arnedesmedt arnedesmedt force-pushed the bugfix/exclusive-properties-dont-allow-booleans-for-open-api-3.1 branch from ec789a7 to e7bf34d Compare November 24, 2023 10:04
@soyuka
Copy link
Member

soyuka commented Nov 24, 2023

nice thanks!

@soyuka soyuka merged commit 58f4a3d into api-platform:3.2 Nov 24, 2023
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.

2 participants