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

Openapi discriminator #91

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

Tails
Copy link

@Tails Tails commented Jul 14, 2021

Work in progress. Adds the discriminator property to the schema extensions for the internally-tagged-enum type. However, doesn't render the internal serde tag yet.

See: #39

@jhoobergs jhoobergs mentioned this pull request Aug 6, 2021
@hatchan
Copy link

hatchan commented Jan 28, 2022

@Tails Hey, I was wondering if you're still going to try and implement this. It seems like a really handy feature.

@Tails
Copy link
Author

Tails commented Jan 29, 2022

I have not yet worked further than the included commits here. I'll get back to it when I have to adjust my OpenAPI schema again.

@ahl
Copy link
Contributor

ahl commented Jun 15, 2022

I'm not sure of the state of this issue, but it seems possible that discriminator may be deprecated at some point: OAI/OpenAPI-Specification#2143

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.

4 participants