-
-
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
[velux] Add an action to simultaneously set main and vane positions #13199
Conversation
Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
…hab-addons into 13049-velux-action
User has confirmed functionality. => Converting from draft to active PR. |
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.
Review all except README
...nhab.binding.velux/src/main/java/org/openhab/binding/velux/internal/action/VeluxActions.java
Outdated
Show resolved
Hide resolved
...nhab.binding.velux/src/main/java/org/openhab/binding/velux/internal/action/VeluxActions.java
Show resolved
Hide resolved
...nhab.binding.velux/src/main/java/org/openhab/binding/velux/internal/action/VeluxActions.java
Show resolved
Hide resolved
...nding.velux/src/main/java/org/openhab/binding/velux/internal/handler/VeluxBridgeHandler.java
Outdated
Show resolved
Hide resolved
...nding.velux/src/main/java/org/openhab/binding/velux/internal/handler/VeluxBridgeHandler.java
Outdated
Show resolved
Hide resolved
...nding.velux/src/main/java/org/openhab/binding/velux/internal/handler/VeluxBridgeHandler.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
...nhab.binding.velux/src/main/java/org/openhab/binding/velux/internal/action/VeluxActions.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
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.
README review
Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
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, thank you
…penhab#13199) * [velux] add moveMainAndVane action * [velux] add claridications to read me * [velux] refactor actions and translate * [velux] fix thing lookup Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch> Signed-off-by: Marcel Goerentz <m.goerentz@t-online.de>
…penhab#13199) * [velux] add moveMainAndVane action * [velux] add claridications to read me * [velux] refactor actions and translate * [velux] fix thing lookup Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
…penhab#13199) * [velux] add moveMainAndVane action * [velux] add claridications to read me * [velux] refactor actions and translate * [velux] fix thing lookup Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
…penhab#13199) * [velux] add moveMainAndVane action * [velux] add claridications to read me * [velux] refactor actions and translate * [velux] fix thing lookup Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch> Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
…penhab#13199) * [velux] add moveMainAndVane action * [velux] add claridications to read me * [velux] refactor actions and translate * [velux] fix thing lookup Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
…penhab#13199) * [velux] add moveMainAndVane action * [velux] add claridications to read me * [velux] refactor actions and translate * [velux] fix thing lookup Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
Background
In PR #12618 we added support for vane/slat position on shutters that have vanes.
On such shutters, if one sends a main position command followed quickly by a vane position command (or vice-versa) while the shutter is still moving to the position given in the first command, then the second command causes the movement of the first command to stop, and starts the movement of the second command.
Requirement
As described in #13049 there is a need to issue simultaneous main and vane position commands from within a rule, so that both positions will be set without the one cancelling the other.
Solution
This PR adds a
moveMainAndVane()
action that can be triggered by a rule to simultaneously set both main position and vane position.Fixes #13049
Note: this PR also includes some extra clarifications in the ReadMe about initial configuration of devices in the KLF200 hub. And concerning the hub's temporary Wi-Fi access point.
Signed-off-by: Andrew Fiddian-Green software@whitebear.ch