Skip to content

Conversation

@Tiller
Copy link

@Tiller Tiller commented Jan 15, 2018

Linked to #2621

  • Creation of a new property implementation: ComposedProperty that contains a field "allOf" listing all the included properties
  • Creation of the associated UT
  • Update of the PropertyDeserializer to handle this new implementation

Note: this PR adds basic support of the functionnality. It probably doesn't handle all the cases of the spec. But I'd say minimal support is better than no support.

…tion of a new Property implementation: ComposedProperty
@webron webron requested a review from frantuma February 12, 2018 22:36
@frantuma frantuma changed the base branch from master to 1.5 April 30, 2018 15:29
@ymohdriz
Copy link
Contributor

ymohdriz commented Jul 2, 2018

Hi @frantuma,

On which swagger-core lib version, this change will be available as it's not available in 1.5.20 version.

Thanks,
Mohammed

@frantuma
Copy link
Contributor

frantuma commented Aug 1, 2018

@ymohdriz @Tiller Thanks for the PR! it's included in #2890 with a small change

@frantuma frantuma closed this Aug 1, 2018
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