-
-
Notifications
You must be signed in to change notification settings - Fork 280
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
docs: clarify the meaning of components #803
docs: clarify the meaning of components #803
Conversation
Co-authored-by: Sergio Moya <1083296+smoya@users.noreply.github.com>
Kudos, SonarCloud Quality Gate passed! |
I don't see the need to rush on this PR so I prefer to wait for 3.0.0 to be honest. |
What can we do to move this forward, is it only reviews we need @fmvilas ? 🙂 |
/rtm |
@@ -157,7 +159,7 @@ Field Name | Type | Description | |||
<a name="A2SServers"></a>servers | [Servers Object](#serversObject) | Provides connection details of servers. | |||
<a name="A2SDefaultContentType"></a>defaultContentType | [Default Content Type](#defaultContentTypeString) | Default content type to use when encoding/decoding a message's payload. | |||
<a name="A2SChannels"></a>channels | [Channels Object](#channelsObject) | **Required** The available channels and messages for the API. | |||
<a name="A2SComponents"></a>components | [Components Object](#componentsObject) | An element to hold various schemas for the specification. | |||
<a name="A2SComponents"></a>components | [Components Object](#componentsObject) | An element to hold various reusable objects for the specification. Everything that is defined inside this object represents a resource that MAY or MAY NOT be used in the rest of the document and MAY or MAY NOT be used by the implemented [Application](#definitionsApplication). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think makes sense to also add this clarification under Components Object
description.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, agree. Would you mind opening the PR?
This PR clarifies the meaning of the Components Object and, more importantly, the meaning of everything outside the Components Object.