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

Use CoAP protocol binding template in examples #1672

Closed
benfrancis opened this issue Aug 17, 2022 · 2 comments
Closed

Use CoAP protocol binding template in examples #1672

benfrancis opened this issue Aug 17, 2022 · 2 comments
Labels
Editorial Issues with no technical impact on implementations PR available Propose closing Problem will be closed shortly if there is no veto. Review for CR transition

Comments

@benfrancis
Copy link
Member

Section 7.2 says:

The following TD example uses a fictional CoAP Protocol Binding, as no such Protocol Binding is available at the time of writing this specification.

Firstly, I think this should say "no such Protocol Binding Template is available" rather than "no such Protocol Binding is available", since a "protocol binding" is defined in a Form in an instance of a Thing Description.

Secondly, as I understand it there is now a CoAP protocol binding template in https://w3c.github.io/wot-binding-templates/bindings/protocols/coap/index.html. Should the example reference this?

@github-actions github-actions bot added the needs-triage Automatically added to new issues. TF should triage them with proper labels label Aug 17, 2022
@sebastiankb sebastiankb added Editorial Issues with no technical impact on implementations and removed needs-triage Automatically added to new issues. TF should triage them with proper labels labels Sep 12, 2022
@JKRhb
Copy link
Member

JKRhb commented Jan 26, 2023

With the merging of #1728, I think this issue is also resolved, right?

@JKRhb JKRhb added the Propose closing Problem will be closed shortly if there is no veto. label Jan 26, 2023
@egekorkan
Copy link
Contributor

Indeed this is fixed to some extent, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Editorial Issues with no technical impact on implementations PR available Propose closing Problem will be closed shortly if there is no veto. Review for CR transition
Projects
None yet
Development

No branches or pull requests

4 participants