-
-
Notifications
You must be signed in to change notification settings - Fork 890
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
Support OpenAPI 3.0 discriminator.mapping #2263
base: master
Are you sure you want to change the base?
Support OpenAPI 3.0 discriminator.mapping #2263
Conversation
…iscriminator is a keyword which points to a property, but never a tag.
@epoberezkin could you check this PR? |
@epoberezkin Any update on this one? I'm about to implement a work-around using extensions and |
I've been running into this issue as well--is this being reviewed/considered for approval? |
I have to use patch as a workaround while it is not approved |
@epoberezkin bump on this! Seems like a feasible solution |
Hi @jeffmay or @VolodymyrMachekhin, could you explain how you applied the work around? |
probably using this: https://www.npmjs.com/package/patch-package |
What issue does this pull request resolve?
resolves #2003
What changes did you make?
option to ignore additional mappings for discriminator. Mappings are not needed in AJV, but they are passed from OAS and should be ignored.
Is there anything that requires more attention while reviewing?
The provided documentation