-
-
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
[tesla] Introduce "allowWakeUpForCommands" option #12184
Conversation
Signed-off-by: Kai Kreuzer <kai@openhab.org>
...ding.tesla/src/main/java/org/openhab/binding/tesla/internal/handler/TeslaAccountHandler.java
Show resolved
Hide resolved
...ding.tesla/src/main/java/org/openhab/binding/tesla/internal/handler/TeslaAccountHandler.java
Show resolved
Hide resolved
@@ -136,6 +136,12 @@ | |||
<label>Allow Wake-Up</label> | |||
<description>Allows waking up the vehicle. Caution: This can result in huge vampire drain!</description> | |||
</parameter> | |||
<parameter name="allowWakeupForCommands" type="boolean" required="false"> | |||
<default>false</default> | |||
<label>Allow Wake-Up For Commands</label> |
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.
Please update the file tesla.properties.
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.
Done, thanks for noticing, I always forget...
Signed-off-by: Kai Kreuzer <kai@openhab.org>
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
* Introduce "allowWakeUpForCommands" option Signed-off-by: Kai Kreuzer <kai@openhab.org>
* Introduce "allowWakeUpForCommands" option Signed-off-by: Kai Kreuzer <kai@openhab.org> Signed-off-by: Nick Waterton <n.waterton@outlook.com>
* Introduce "allowWakeUpForCommands" option Signed-off-by: Kai Kreuzer <kai@openhab.org>
* Introduce "allowWakeUpForCommands" option Signed-off-by: Kai Kreuzer <kai@openhab.org>
* Introduce "allowWakeUpForCommands" option Signed-off-by: Kai Kreuzer <kai@openhab.org> Signed-off-by: Andras Uhrin <andras.uhrin@gmail.com>
* Introduce "allowWakeUpForCommands" option Signed-off-by: Kai Kreuzer <kai@openhab.org>
This introduces a new configuration option to the vehicle thing, which allows the binding to wake up the vehicle in case that commands are sent to it.
This is specifically handy for things like activating the auto-conditioning, Sentry mode or similar things.
As waking up the vehicle usually takes ~15 seconds, sending the command is retried 3 times (with 15 secs pause) until it succeeds. Naturally, this means that the command is not executed immediately, but that it is delayed by a considerable time.
Signed-off-by: Kai Kreuzer kai@openhab.org