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

Reference schema properties from another schema #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

IlonaShishov
Copy link

@IlonaShishov IlonaShishov commented Jan 27, 2022

Signed-off-by: Ilona Shishov [email protected]

Hi,
While implementing the following APIs, I found that model 'RetrieveStatusResponse' was missing the 'CustomerRelationshipStatus' property.
Also I found that a conversion from openapi to protobuf-schema using openapi-generator results in incorrect auto referencing to other unrelated schemas due to similar property groups.
As a solution to this behavior, I suggest to replace the property groups inside several schemas with a reference to a base schema.

Best regards,
Ilona Shishov

@arnabbian
Copy link
Collaborator

Signed-off-by: Ilona Shishov [email protected]

While implementing the following APIs, I found that model 'RetrieveStatusResponse' was lacking the 'CustomerRelationshipStatus' property. Also, a conversion from openapi to protobuf-schema using openapi-generator results in incorrect auto referencing to other unrelated schemas due to similar property groups. As a solution to this behavior, I suggest to replace the property groups inside several schemas with a reference to a base schema.

Hi,
Many thanks for you feedback. We are considering this and will get back to you soon.

Kind regards,
BIAN

@IlonaShishov
Copy link
Author

Signed-off-by: Ilona Shishov [email protected]
While implementing the following APIs, I found that model 'RetrieveStatusResponse' was lacking the 'CustomerRelationshipStatus' property. Also, a conversion from openapi to protobuf-schema using openapi-generator results in incorrect auto referencing to other unrelated schemas due to similar property groups. As a solution to this behavior, I suggest to replace the property groups inside several schemas with a reference to a base schema.

Hi, Many thanks for you feedback. We are considering this and will get back to you soon.

Kind regards, BIAN

Thank you

@arnabbian
Copy link
Collaborator

Signed-off-by: Ilona Shishov [email protected]
While implementing the following APIs, I found that model 'RetrieveStatusResponse' was lacking the 'CustomerRelationshipStatus' property. Also, a conversion from openapi to protobuf-schema using openapi-generator results in incorrect auto referencing to other unrelated schemas due to similar property groups. As a solution to this behavior, I suggest to replace the property groups inside several schemas with a reference to a base schema.

Hi, Many thanks for you feedback. We are considering this and will get back to you soon.
Kind regards, BIAN

Thank you

Hi Ilona,
We have now reviewed this Pull Request at BIAN.

We agree with your proposed changes, and this will be updated in our Core BIAN model, and available in the next BIAN Release.

For the time being, we recommend you progress with this change in your implementations, as necessary, and the next Release of BIAN will reflect this change.

Many thanks for your feedback, and please feel free to get back in touch with any other points you would like to raise.

Kind regards,
BIAN

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.

2 participants