search

ReneNulschDE / mbapi2020

Custom Component to integrate MercedesME devices into Home-Assistant
Other
157 stars 30 forks source link

Add chargeflapacstatus #237

Closed Thomas55555 closed 4 months ago

Thomas55555 commented 4 months ago

Hi, ich hoffe, es ist okay, wenn ich auf deutsch schreibe. Mit dem PR soll ein binary sensor chargeflapacstatus hinzugefügt werden. Beim schreiben ist mir aufgefallen, dass die Invertier-Funktion nicht funktioniert. Habe deswegen auch noch was hinzugefügt.

ReneNulschDE commented 4 months ago

Warum änderst Du den parkbrakestatus sensor?

Thomas55555 commented 4 months ago

Das war nur zum testen, ob die flip Funktion funktioniert. Ich muss dann immer zwischen meinem Entwicklungscontainer und dem git Verzeichnis hin und her kopieren. Da geht öfters mal was schief. Wie programmierst du, in dem repository, so dass du auch direkt in github commiten kannst?

ReneNulschDE commented 4 months ago

Wie programmierst du

Ich habe mehrere HA Installationen (komplett leer geräumt, ohne default_config und sonst irgendwas) mit den letzten 3 HA Versionen und eine Installation mit dem aktuellen Code aus dem HA-Core repo. Dort habe ich per "soft/symbolic link" den dev ordner in custom_components eingebunden. Damit kann ich immer direkt alle möglichen Versionen durchtesten.

Das war nur zum testen,

Löscht Du das noch, dann merge ich.

Thomas55555 commented 4 months ago

Ist gelöscht. Aber mir ist jetzt noch aufgefallen, dass egal ob ich bei der Bremse True oder False habe, es keine Auswirkungen auf den Status hat. Bei der Ladeklappe kommt es aber an.

ReneNulschDE commented 4 months ago

Ich merge es erstmal und teste es morgen mal mit dem Auto...

ReneNulschDE commented 4 months ago

Dein Parkbreak Change war richtig, sorry... Hab es wieder eingebracht.