search

tesla-local-control / tesla-local-control-addon

Control your Tesla locally from HomeAssistant
Apache License 2.0
21 stars 2 forks source link

[v0.0.9] Manage simple errors in tesla-control #40

Closed raphmur closed 3 months ago

raphmur commented 3 months ago

Capture stdout & stderr to identify errors.

So far only messages starting with Failed to execute command: car could not execute command are processed. In this case, the car has received the command, it is valid, but for a reason (not processed) cannot execute it. Eg. is already charging, charge limit already set, AC already on, ...

Site commit: warn on standalone broken for now

epenet commented 3 months ago

Could the version be added to the MQTT device information?

raphmur commented 3 months ago

Could the version be added to the MQTT device information?

You mean adding the add-on version as an MQTT entity?

EDIT: ok I get it, like this maybe. Do you know how to do (quicker asking than searching ;) )

image

epenet commented 3 months ago

I think it should be straightforward adding sw_version to https://github.com/tesla-local-control/tesla-local-control-addon/blob/a44f6b561decc03bdfc269ba5cbf2640b16ca752/tesla_ble_mqtt/rootfs/app/discovery.sh#L8-L13

raphmur commented 3 months ago

Ok great thanks. Can you open an issue? Or even a PR... :)

epenet commented 3 months ago

I opened issue #41 Until I can get standalone working I cannot test any of my changes so I don't feel confortable opening PRs.

raphmur commented 3 months ago

I opened issue #41 Until I can get standalone working I cannot test any of my changes so I don't feel confortable opening PRs.

Ok thanks, At least we track your idea. Standalone should be working by the end of the week latest

raphmur commented 3 months ago

we all revied. I am force pushing this one.