-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
discriminator field #160
Comments
Welcome to AsyncAPI. Thanks a lot for reporting your first issue. Please check out our contributors guide and the instructions about a basic recommended setup useful for opening a pull request. |
any suggestion or help here |
@fcmdeveloper1 you need to head over to the spec repo for proposing spec changes 🙂 Instead of adapting the AsyncAPI Schema, have you considered just using the OpenAPI schema instead by altering the schema format 🤔? |
Yep, agree with you. I can't change That's why I reject your proposal to change it to OpenAPI schemaObject.discriminator You can describe your scenario, so we could help you to achieve what you want |
thanks @Pakisan for your help and support, my case was that i have some objects has inheritance and i need to use them in Kafka consumer which creates the discriminator as object from the swagger, my case is same like the description in this link i used springwolf, but the generated discriminator doesn't match what asyncAPI is expecting, and i am not sure how to do the required formatting yet. thanks for your help and support |
@fcmdeveloper1 hi! Can you check this response - asyncapi/spec#1037 (comment)? |
Hi Pakisan,
you can use the below object to test the behavior: { My Best Regards |
@fcmdeveloper1 hi! Will check it. Good news, you can checkout branch |
Dear @Pakisan, so, i believe the APIs like the springwolf and others needs to do the big work to apply these mappings/differences which i believe asyncAPI and openAPI must be unified in a specification to make a standard. again, i am not sure if a creation of schema definition like this one "application/vnd.oai.openapi;version=3.0.0" can help to unify this or not. Async API specs: open API specs discriminator: may be we can have additional discussions to figure out a way of this, let me know if i can support. thanks a lot for your support. My Best Regards |
This issue has been automatically marked as stale because it has not had recent activity 😴 It will be closed in 120 days if no further activity occurs. To unstale this issue, add a comment with a detailed explanation. There can be many reasons why some specific issue has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model. Let us figure out together how to push this issue forward. Connect with us through one of many communication channels we established here. Thank you for your patience ❤️ |
can we change the specs, so, it can follow the open API specs
instead of having discriminator as string
to follow swagger/openAPI 3 to have it like
discriminator:
propertyName: type
or is there anything in the APIs let me do this mapping
The text was updated successfully, but these errors were encountered: