evcc
evcc copied to clipboard
Replace resetOnDisconnect by onDisconnect configuration
@andig: Hier noch mein Vorschlag zu https://github.com/evcc-io/docs/issues/39#issuecomment-1022040792
Freue mich über ein Feedback. Wenn wir eine Lösung gefunden haben, kann ich gerne noch weitere Testcases hinzufügen.
Improvements
- user can control which parameters shall be reset on disconnect
- values from onDisconnect configuration are applied as defaults on startup, therefore separate configuration of those loadpoint parameters (e.g. mode) is no more necessary
- loadpoint configuration is aligned with vehicle.onIdentify
Ich habs mal rebased
@rivengh ich habe in https://github.com/evcc-io/evcc/pull/3860 angefangen einige Tests für die ganzen Modi zu ergänzen. Ich finde den PR hier immer noch super, bin aber noch nicht sicher, inwiefern wir die Optionen duplizieren sollten.
/cc @premultiply was meinst Du?
Eine weitere Frage wäre, ob onDisconnect
in Anbetracht der manuellen Fahrzeugauswahl jetzt tatsächlich beim Disconnect erfolgen sollte, oder auch wenn aufs Gastfahrzeug gewechselt wird?
@rivengh danke nochmal für Deinen PR. Der Bedarf müsste sich zwischenzeitlich mit dem Umbau der Fahrzeugkonfiguration erledigt haben?
Scheint sich erledigt zu haben- vielen Dank! Falls noch ein Thema bitte melden.
Der Bedarf müsste sich zwischenzeitlich mit dem Umbau der Fahrzeugkonfiguration erledigt haben?
Ja, definitiv. Vielen Dank!