Skip to content

support @SecurityScheme/Fix OpenAPI Serializer #164

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

Merged
merged 12 commits into from
Mar 1, 2023

Conversation

SentryMan
Copy link
Collaborator

@SentryMan SentryMan commented Feb 20, 2023

all the changes of #163 + fixes to the openAPI serialization to not get into a recursive loop on the new openAPI enums added in the latest swagger version

@SentryMan
Copy link
Collaborator Author

Yeah, I'm good with this. @rbygrave please review when you get time.

@SentryMan SentryMan changed the title support @SecurityScheme/ Fix OpenAPI Serializer support @SecurityScheme/Fix OpenAPI Serializer Feb 28, 2023
@SentryMan SentryMan mentioned this pull request Mar 1, 2023
@rbygrave rbygrave added this to the 1.28 milestone Mar 1, 2023
@rbygrave
Copy link
Contributor

rbygrave commented Mar 1, 2023

Very nice !!

@rbygrave rbygrave merged commit 1f47591 into avaje:master Mar 1, 2023
@SentryMan SentryMan deleted the serializer branch March 1, 2023 23:07
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