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

Add OpenAPI Specification v3.1 objects #41

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

tyzhnenko
Copy link

No description provided.

@RobertoPrevato
Copy link
Member

Hi @tyzhnenko
Thank You for your contribution! It seems there is more analysis needed because I see several properties that were removed from OpenAPI specification in v3 are still included in the classes you proposed in this PR.

By the way - the complexity of OAD makes me feel that supporting it fully in web frameworks is a work for people like Hermes from Futurama! 😁

openapidocs/v31.py Outdated Show resolved Hide resolved
openapidocs/v31.py Outdated Show resolved Hide resolved
openapidocs/v31.py Outdated Show resolved Hide resolved
openapidocs/v31.py Show resolved Hide resolved
@tyzhnenko
Copy link
Author

@RobertoPrevato I'll take a look at your comments and make changes. Thanks to the review

Remove example from MediaType
Remove nullable from Schema
Add support list in type in Schema
Add content_encoding to Schema
Add content_media_type to Schema
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