-
-
Notifications
You must be signed in to change notification settings - Fork 31
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
[items] Refactor metadata
& itemchannllink
APIs
#212
Conversation
metadata
& itemchannllink
APIs
5c098d5
to
ab1ecfe
Compare
@rkoshak Ping. |
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.
Their old versions have been removed (since metadata was unable to support configuration, I'm not sure how many even used it).
So i poked around a little on this, i think the few that were using item links, used the native Java proxy objects and did not use this library, so i think its probably a very small population if any .
I had only a few small nits, other wise LGTM
@digitaldan I have addressed your review, can you please have a look at the changes?
I thing the same, but I'll label it as breaking to make sure that no one is suprised |
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 had two more code style changes recommendations. Feel free to merge either way. Thanks!
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com> Co-authored-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com> Co-authored-by: Dan Cunningham <dan@digitaldan.com>
Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
f403da7
to
6bed6a7
Compare
Sorry I missed the ping. It all looks as I would expect from the metadata side (I never messed with channel links in rules so don't have much to say there. |
Been playing around with this. Would be nice if I could remove all channel links or metadata by excluding the name or some other wildcard. Sometimes I don't want to specify the name or channel link, I just want to remove/replace. |
IIRC, openHAB Core provides that possibility, so I could add this to the library as well. Can you please open an issue and describe your feature request? |
Fixes #189.
Description
This PR completely refactors the
metadata
and theitemchannellink
APIs.Their old versions have been removed (since metadata was unable to support configuration, I'm not sure how many even used it).
The new
metadata
API is available under theitems
namespace and is able to add and change metadata configuration as well.itemchannellink
got a few internal improvements and a renaming.To Do
Testing
The following script tests all methods of these APIs: