-
Notifications
You must be signed in to change notification settings - Fork 5
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
Deprecate ObservableProperty, ActuatableProperty #237
Comments
@alexrobin @KathiSchleidt @ldesousa any comment on this? |
I am in favour of this change. |
Kathi also agreed earlier |
fine with me as well |
I support this as well |
@alexrobin @sgrellet could you review #249 please? |
The only thing I noticed is this part of the documentation of
Same thing in the RDF:
However, should the Otherwise, it looks good to me. |
Right. That restriction is 'inherited' from the parent class @alexrobin @sgrellet could you approve review so it can be merged. |
Only
sosa:Property
is needed.No functional difference between the sub-classes.
Legacy sub-class relationships in ssn-deprecated.ttl would preserve backward compatibility.
See argument here: #235 (comment)
The text was updated successfully, but these errors were encountered: