tesla-local-control / tesla_ble_mqtt_core

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

[feat] Auto MAC addr detection #86

Closed baylanger closed 3 months ago

raphmur commented 3 months ago

See https://github.com/tesla-local-control/tesla-local-control-addon/pull/32

Anyways, referencing this issue that will be solved with this: https://github.com/tesla-local-control/tesla-local-control-addon/issues/31

baylanger commented 3 months ago

Excellent work and a very useful new feature, many thanks A lot of changes. Have either of you tested it before we merge into #main? I could create a _docker branch and use it to test, however I might not complete it today

Yes I have been testing throughout the dev process but more tester is always good. I did create the addon PR but not for Docker. Will do if you don't have time and that I get the time.

iainbullock commented 3 months ago

Ok shall we merge into core? I don't think there's a lot to do for _docker from what I can see. Eg Remove BLE_MAC_LIST references Update installation docs Update CHANGELOG

Anything else? I can do that later no problem