Skip to content

Don't configure the parent with allOf #246

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

Merged
merged 1 commit into from
Jun 28, 2016
Merged

Conversation

cbornet
Copy link
Contributor

@cbornet cbornet commented May 19, 2016

Fix #245

cbornet added a commit to cbornet/swagger-codegen that referenced this pull request May 30, 2016
If a composed model (allOf) doesn't have any parent and one of its interface has a discriminator field, then set this interface as parent.

See swagger-api#2096
See swagger-api/swagger-parser#246
@jeff9finger
Copy link

+1

@cbornet
Copy link
Contributor Author

cbornet commented Jun 3, 2016

rebased to resolve conflict.

@cbornet
Copy link
Contributor Author

cbornet commented Jul 5, 2016

@fehguy Thanks for merging 👍 !
Any chance to see a release soon ? This would allow to fix the Java inheritance issues in swagger-codegen !

@fehguy
Copy link
Contributor

fehguy commented Jul 7, 2016

@cbornet very close, this is required for the codegen release, which we (and you!) are working on.

cbornet added a commit to cbornet/swagger-codegen that referenced this pull request Jul 15, 2016
If a composed model (allOf) doesn't have any parent and one of its interface has a discriminator field, then set this interface as parent.

See swagger-api#2096
See swagger-api/swagger-parser#246
fehguy pushed a commit to swagger-api/swagger-codegen that referenced this pull request Jul 17, 2016
If a composed model (allOf) doesn't have any parent and one of its interface has a discriminator field, then set this interface as parent.

See #2096
See swagger-api/swagger-parser#246
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.

3 participants