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

Update SchemaGenerator to fix #2514 and #2052 #2683

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

danielcrabtree
Copy link

@danielcrabtree danielcrabtree commented Jul 15, 2023

This pull request fixes (reverts) the placement of the discriminator to the polymorphic oneOf element. Fixes #2052.

This pull request also fixes #2514 and the bug where enabling UseOneOfForPolymorphism forces UseAllOfForInheritance=true (when UseAllOfForInheritance=false).

@dgwaldo
Copy link

dgwaldo commented Jan 31, 2024

Sure hope this gets merged some day :)

@Havunen
Copy link

Havunen commented Feb 12, 2024

You can port this to my fork https://github.com/Havunen/DotSwashbuckle if you are interested about this feature, however some tests need to be added

@martincostello
Copy link
Collaborator

Thanks for contributing - if you'd like to continue with this pull request, please rebase against the default branch to pick up our new CI. Please also add some tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants