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

xml object and root schemas #57

Closed
MikeRalphson opened this issue Jan 22, 2018 · 2 comments
Closed

xml object and root schemas #57

MikeRalphson opened this issue Jan 22, 2018 · 2 comments
Labels

Comments

@MikeRalphson
Copy link
Contributor

The xml example/test document uses the xml object on a root schema.

https://github.com/asyncapi/asyncapi/blob/70bb40a42a463917a963e544ae4b2395bf40b22e/test/docs/xml.yaml#L29-L32

From the spec.

This MAY be used only on properties schemas. It has no effect on root schemas. Adds Additional metadata to describe the XML representation format of this property.

It may be worth following the discussion here to see how a similar issue gets resolved in the OAS, or simply clarifying the wording in AsyncAPI to allow this.

@fmvilas
Copy link
Member

fmvilas commented Jan 22, 2018

You're right. In my opinion, it would make sense to have it there. Let's wait for the OAS guys to see what they say, so we're aligned. Thanks for pointing out!

@stale
Copy link

stale bot commented Oct 29, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Oct 29, 2019
@stale stale bot closed this as completed Nov 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants