-
Notifications
You must be signed in to change notification settings - Fork 23
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
[deconz] Added support for publishing ChannelDescriptionChangedEvents #138
[deconz] Added support for publishing ChannelDescriptionChangedEvents #138
Conversation
...g.deconz/src/main/java/org/smarthomej/binding/deconz/internal/handler/GroupThingHandler.java
Show resolved
Hide resolved
...org.smarthomej.binding.deconz/src/main/java/org/smarthomej/binding/deconz/internal/Util.java
Outdated
Show resolved
Hide resolved
Can you rebase on 3.2.x? We can't merge this to main because the new methods and fields are not available in OHC 3.0.x. I tried just changing the base branch but that resulted in a very big changeset. |
c09c42a
to
e6f6af2
Compare
Done. |
bundles/org.smarthomej.binding.deconz/src/main/java/org/smarthomej/binding/deconz/internal/DeconzDynamicStateDescriptionProvider.java:[77,55] Potential null pointer access: this expression has a '@nullable' type @cweitkamp, can you fix that? |
Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> Use name for value and label of CommandOption Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de> Incorporated comments from review Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
Signed-off-by: Christoph Weitkamp <github@christophweitkamp.de>
37dae6d
to
153aec8
Compare
Done. |
ChannelDescriptionChangedEvents
Related to openhab/openhab-core#1505
Depends on openhab/openhab-core#2410
Signed-off-by: Christoph Weitkamp github@christophweitkamp.de