-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[somfytahoma] New method to send a command to several devices in the same place #10347
Conversation
…same place Also include the place into the inbox label (discovery) Signed-off-by: Laurent Garnier <lg.hc@free.fr>
...java/org/openhab/binding/somfytahoma/internal/discovery/SomfyTahomaItemDiscoveryService.java
Outdated
Show resolved
Hide resolved
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.
Generally I am asking myself why we should have this code increment, if it does not bring any new channel, so there is no added value of this code for users. Sorry if I am missing something, but I would at lease expect a way how to invoke this method sendCommandToSameDevicesInPlace
I try to provide small PR to move to my final PR for Cozytouch integration. |
Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Note that you will be able later to enable the "room" feature for certain Tahoma devices the same way I do it for Cozytouch radiators. But it has to be thought (by you) on what devices it makes really sense. |
OK, I understand |
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.
LGTM
@fwolter : please review |
@openhab/add-ons-maintainers : please review & merge. |
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.
LGTM
…same place (openhab#10347) * [somfytahoma] New method to send a command to several devices in the same place Also include the place into the inbox label (discovery) Signed-off-by: Laurent Garnier <lg.hc@free.fr> * Review comment: remove unnecessary method Signed-off-by: Laurent Garnier <lg.hc@free.fr> Signed-off-by: John Marshall <john.marshall.au@gmail.com>
…same place (openhab#10347) * [somfytahoma] New method to send a command to several devices in the same place Also include the place into the inbox label (discovery) Signed-off-by: Laurent Garnier <lg.hc@free.fr> * Review comment: remove unnecessary method Signed-off-by: Laurent Garnier <lg.hc@free.fr>
…same place (openhab#10347) * [somfytahoma] New method to send a command to several devices in the same place Also include the place into the inbox label (discovery) Signed-off-by: Laurent Garnier <lg.hc@free.fr> * Review comment: remove unnecessary method Signed-off-by: Laurent Garnier <lg.hc@free.fr>
…same place (openhab#10347) * [somfytahoma] New method to send a command to several devices in the same place Also include the place into the inbox label (discovery) Signed-off-by: Laurent Garnier <lg.hc@free.fr> * Review comment: remove unnecessary method Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Also include the place into the inbox label (discovery)
Signed-off-by: Laurent Garnier lg.hc@free.fr