-
Notifications
You must be signed in to change notification settings - Fork 25
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
Reorganize intro and section 4 #266
Conversation
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.
Just two minor points below.
index.html
Outdated
</ul> | ||
Description[[WOT-THING-DESCRIPTION]] (TD), with Things that exhibit diverse protocols, payload formats and a | ||
combination of these in platforms and frameworks. | ||
The mechanism that allows this is called the Binding Mechanism, without which TDs could not build Hypermedia Controls |
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.
allow this what?
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 have added some text but it is a bit difficult to express this in a short/elegant way
Co-authored-by: Cristiano Aguzzi <relu91@users.noreply.github.com>
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'd say this is a great improvement in the understandability of our binding mechanism. Well done!
Just one minor point, using the verb to form
in the following sentence feels weird to me:
This results in different protocols and media types and forms the central challenge for the Web of Things
It might be good to get a native speaker's opinion on this.
Call of 29.03: No objections, @egekorkan will fix the definition problem and merge |
fixes #265 , fixes #238
Additionally, it does the following:
Preview | Diff