Skip to content
This repository has been archived by the owner on Feb 17, 2024. It is now read-only.

Remove discriminator from any union type definition #307

Closed
sichvoge opened this issue Mar 14, 2016 · 1 comment
Closed

Remove discriminator from any union type definition #307

sichvoge opened this issue Mar 14, 2016 · 1 comment
Assignees
@sichvoge
Milestone
RAML 1.0

Comments

@sichvoge
Copy link
Contributor

In RAML 1.0 RC1 you are able to define a discriminator for union types. This could introduce a lot of ambiguity when, for example, the types itself have their definition of a discriminator.

To avoid any ambiguity, the suggestion is to remove discriminator from any union type.

PersonOrDog: 
   type: Person | Dog
   discriminator: hasTail # will be removed in RC2
@sichvoge sichvoge self-assigned this Mar 14, 2016
sichvoge added a commit that referenced this issue Mar 23, 2016
@sichvoge
discriminators are also not allowed in any union types according to #307
251752a
@sichvoge sichvoge mentioned this issue Mar 23, 2016
Merged
@sichvoge
Copy link
Contributor Author

sichvoge commented Apr 4, 2016

Not done! It is still there and was missing. Reopen it until fixed.

@sichvoge sichvoge reopened this Apr 4, 2016
@sichvoge sichvoge mentioned this issue Apr 4, 2016
Merged
@sichvoge sichvoge closed this as completed Apr 4, 2016
@petrochenko-pavel-a petrochenko-pavel-a mentioned this issue Apr 7, 2016
Closed
@VasiliyLysokobylko VasiliyLysokobylko mentioned this issue Apr 7, 2016
Closed
@sichvoge sichvoge added this to the RAML 1.0 milestone May 1, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@sichvoge sichvoge

Labels
None yet
Milestone
RAML 1.0
Development

No branches or pull requests
1 participant
@sichvoge