evcc icon indicating copy to clipboard operation
evcc copied to clipboard

Replace resetOnDisconnect by onDisconnect configuration

Open rivengh opened this issue 2 years ago • 3 comments

@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

rivengh avatar Feb 13 '22 17:02 rivengh

Ich habs mal rebased

andig avatar Mar 17 '22 16:03 andig

@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?

andig avatar Jul 15 '22 16:07 andig

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?

andig avatar Jul 15 '22 17:07 andig

@rivengh danke nochmal für Deinen PR. Der Bedarf müsste sich zwischenzeitlich mit dem Umbau der Fahrzeugkonfiguration erledigt haben?

andig avatar Oct 12 '22 13:10 andig

Scheint sich erledigt zu haben- vielen Dank! Falls noch ein Thema bitte melden.

andig avatar Oct 14 '22 10:10 andig

Der Bedarf müsste sich zwischenzeitlich mit dem Umbau der Fahrzeugkonfiguration erledigt haben?

Ja, definitiv. Vielen Dank!

rivengh avatar Oct 17 '22 09:10 rivengh