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

Do we need an "observeData" field? #65

Closed
mmccool opened this issue Nov 7, 2017 · 1 comment
Closed

Do we need an "observeData" field? #65

mmccool opened this issue Nov 7, 2017 · 1 comment

Comments

@mmccool
Copy link
Contributor

mmccool commented Nov 7, 2017

Currently we have "outputData" and "inputData", corresponding to payloads for read and write operations or inputs and outputs from Actions, or output data from Events.

However, if we add an "observe" capability to Properties, then there is no guarantee that a given protocol will use the same payload format as a read. As an example, the Mozilla proposal wraps the payload for an observe message in an additional JSON object that includes a timestamp, so the structure (not just the fields) are different from that returned from a GET. This example shows that we can't necessarily just ignore fields to deal with differences, as the structure itself might be different.

So, we need a separate way to give payloads for observe events. One suggestion that generalizes inputData and outputData to multiple roles is given in issue #66. Of course we could also just add an "observeData" tag.

@mkovatsc
Copy link
Contributor

If the "observe data" is different from the Property data, I don't see any point in forcing this into the same Property. It is simply an Event that sends data that is different from the Property.

@mkovatsc mkovatsc closed this as completed Feb 9, 2018
jmcanterafonseca pushed a commit to jmcanterafonseca/wot-thing-description that referenced this issue May 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants