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
[openwebnet] add support for WHO=2 Automation #8437
Conversation
… 0.3.0. Fixes openhab#8436 Signed-off-by: Massimo Valla <mvcode00@gmail.com>
Travis tests were successfulHey @mvalla, |
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.
Just some comments mainly about reducing log levels. Regarding logging see also https://www.openhab.org/docs/developer/guidelines.html#f-logging
...webnet/src/main/java/org/openhab/binding/openwebnet/handler/OpenWebNetAutomationHandler.java
Outdated
Show resolved
Hide resolved
...webnet/src/main/java/org/openhab/binding/openwebnet/handler/OpenWebNetAutomationHandler.java
Outdated
Show resolved
Hide resolved
...webnet/src/main/java/org/openhab/binding/openwebnet/handler/OpenWebNetAutomationHandler.java
Outdated
Show resolved
Hide resolved
...webnet/src/main/java/org/openhab/binding/openwebnet/handler/OpenWebNetAutomationHandler.java
Outdated
Show resolved
Hide resolved
...webnet/src/main/java/org/openhab/binding/openwebnet/handler/OpenWebNetAutomationHandler.java
Show resolved
Hide resolved
...webnet/src/main/java/org/openhab/binding/openwebnet/handler/OpenWebNetAutomationHandler.java
Outdated
Show resolved
Hide resolved
...webnet/src/main/java/org/openhab/binding/openwebnet/handler/OpenWebNetAutomationHandler.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Massimo Valla <mvcode00@gmail.com>
Thanks for the suggestions and quick review. I changed where requested and also minimized all INFO/WARN log messages. |
Travis tests were successfulHey @mvalla, |
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. Will merge once build is finished.
* update openwebnet4j to 0.3.0. * Closes openhab#8436 Signed-off-by: Massimo Valla <mvcode00@gmail.com>
* update openwebnet4j to 0.3.0. * Closes openhab#8436 Signed-off-by: Massimo Valla <mvcode00@gmail.com>
* update openwebnet4j to 0.3.0. * Closes openhab#8436 Signed-off-by: Massimo Valla <mvcode00@gmail.com>
Add support for WHO=2 Automation.
Updated openwebnet4j to 0.3.0.
Fixes #8436