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.

Sign up for GitHub

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

Jump to bottom

Properties in ComposedModel #2083

Closed
beikov opened this issue Apr 28, 2024 · 4 comments
Closed

Properties in ComposedModel #2083

beikov opened this issue Apr 28, 2024 · 4 comments

Comments

@beikov
Copy link

beikov commented Apr 28, 2024

The Azure OpenAPI specification uses "properties" in definitions that also have allOf i.e. are a ComposedModel.

I don't know if this is valid or not, but Swagger should support parsing this.
beikov added a commit to beikov/swagger-parser that referenced this issue Apr 28, 2024
@beikov
Test and fix swagger-api#2083 properties in ComposedModel
b3b2c78
@beikov beikov mentioned this issue Apr 28, 2024
Merged
beikov added a commit to beikov/swagger-parser that referenced this issue May 2, 2024
@beikov
Test and fix swagger-api#2083 properties in ComposedModel
2977961
frantuma pushed a commit that referenced this issue May 2, 2024
@beikov @frantuma
Test and fix #2083 properties in ComposedModel
8a4473d
@frantuma
Copy link
Member

frantuma commented May 2, 2024

fixed by #2084

@frantuma frantuma closed this as completed May 2, 2024
@beikov
Copy link
Author

beikov commented May 2, 2024

Awesome, thanks so much @frantuma! I don't want to rush you, but do you have a release planned soon? I'd like to get past my SNAPSHOT dependency :)

@frantuma
Copy link
Member

frantuma commented May 2, 2024

@beikov released 1.0.71

@beikov
Copy link
Author

beikov commented May 2, 2024

Thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@beikov @frantuma