-
-
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
[emotiva] Fix missing data in source channels #17365
base: main
Are you sure you want to change the base?
Conversation
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.
Thanks, left two minor comments.
Fine to merge for 4.3.0, for backporting, it would be nice to also have a user confirmed test to minimalise the risks for regression.
...ing.emotiva/src/main/java/org/openhab/binding/emotiva/internal/EmotivaUdpSendingService.java
Show resolved
Hide resolved
...ing.emotiva/src/main/java/org/openhab/binding/emotiva/internal/EmotivaUdpSendingService.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Espen Fossen <[email protected]>
98b35a0
to
365ba9d
Compare
Fixed the minor things. Where you thinking of an unit test or an actual user test? |
Depends on the extend of the unit tests. Add tests if you can and will see, |
@espenaf do you plan to add unit tests to this PR? |
I have started looking into it, but discovered I need to refactor a few things to be able to write some proper tests for this. Hopefully I will get it done over the weekend, but it could also be a separate PR to not cram to many changes into this fix. |
Fixes #17363
Can be backported to 4.2.x