-
-
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
[hueemulation] Fix xy change response #10692
Conversation
4772c36
to
9c2480a
Compare
....openhab.io.hueemulation/src/main/java/org/openhab/io/hueemulation/internal/ConfigStore.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.
Can you update the switchOnWithXY test in LightsAndGroupsTests to explicitly check the response contains "xy"
Signed-off-by: Flole <[email protected]>
@davidgraeff Could you please review this real quick? |
@digitaldan Maybe you can review this? It's just a small change, shouldn't take long. |
@kaikreuzer As nobody else seems to want to review this PR, maybe you could do it? It's a rather simple change, so I have no clue why it's taking so long.... |
Signed-off-by: Flole <[email protected]> Signed-off-by: Nick Waterton <[email protected]>
Signed-off-by: Flole <[email protected]> Signed-off-by: Michael Schmidt <[email protected]>
Signed-off-by: Flole <[email protected]>
Signed-off-by: Flole <[email protected]>
Signed-off-by: Flole <[email protected]>
Signed-off-by: Flole <[email protected]> Signed-off-by: Andras Uhrin <[email protected]>
Fixes the following problem mentioned in #10691 :