search

tesla-local-control / tesla_ble_mqtt_core

Core content for the tesla_ble tools
Apache License 2.0
3 stars 3 forks source link

Fix charging set amps log #116

Closed g4rb4g3 closed 2 months ago

g4rb4g3 commented 2 months ago

Thanks for this great project, works pretty well for me! 😃 One thing that confused me was this line in the logs: sending Set charging Amps to 5A then to 3 to vin:ASDF1234567890 command:charging-set-amps 3 since I did not request 5A charging current. Checking the source revealed that only logs are confusing and everything works as expected. So this PR improves on logging so others might end up less confused. 😂

raphmur commented 2 months ago

Thank for the contribution. Indeed, this is better logging. I will set for merge on the stage branch which collects all for next release