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

arch-property-readable : The state exposed by a Property MUST be retrievable (readable). #633

Closed
Tracked by #625
mlagally opened this issue Nov 10, 2021 · 4 comments · Fixed by #655
Closed
Tracked by #625

Comments

@mlagally
Copy link
Contributor

No description provided.

@mlagally mlagally mentioned this issue Nov 10, 2021
10 tasks
@benfrancis
Copy link
Member

benfrancis commented Nov 16, 2021

From #625 (comment)...

Conclusion: ❌ I wouldn't mind if this was true, but it clearly isn't true because there's a writeOnly member of PropertyAffordance.


Suggestion: Remove the assertion.

@sebastiankb
Copy link
Contributor

There can be use cases that properties are defined only for writable purposes (eg, parameter settings) and other properties for readonly (e.g., read latest parameter settings). I think, it is not best practice and not really REST style. However, it does not hurt and at least would open the opportunity to retrofit existing implementations.

btw: I would be open to have a recommendation for the profile that says properties are always readable .

@mlagally
Copy link
Contributor Author

arch call on 25.11.:
remove here, clarify in TD, consider mandating in profile

@sebastiankb
Copy link
Contributor

PR clarifies the usage of readOnly and writeOnly in Properties in the TD spec w3c/wot-thing-description#1306 (review)

Assertation in Arch can be removed

sebastiankb added a commit that referenced this issue Dec 16, 2021
sebastiankb added a commit that referenced this issue Dec 16, 2021
sebastiankb added a commit that referenced this issue Dec 16, 2021
sebastiankb added a commit that referenced this issue Dec 16, 2021
mlagally added a commit that referenced this issue Dec 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants