-
-
Notifications
You must be signed in to change notification settings - Fork 704
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add loadpoint config api (BC) #12958
base: master
Are you sure you want to change the base?
Conversation
@andig die APIs sehen gut aus. Funktioniert alles wie es soll. Wir hatten ja gesagt, wir wollten den Scope klein halten und Vehicle/Charger erstmal ausklammern. Ich glaub es ist doch ne gute Idee, wenn wir die Referenz-Felder hier gleich mit aufnehmen. Sonst haben wir einen komischen Zwischenstand den man schwer releasen kann. Magst du die beiden Felder ergänzen? |
Mache ich. Am Fahrzeug könnten wir dann noch |
So, jetzt nochmal langsam. Was meinst Du damit konkret? Phases z.B. ist doch drin? |
Mit Referenz Felder meine ich:
Da stehen dann die 'name's der am loadpoint verknüpften devices drin und sind darüber auch änderbar. |
Ah, ok. Aber erstmal ohne Updatefähigkeit? |
Wie es passt. Update wäre cool (bspw default Fahrzeug). Lesen wäre aber auch ein Fortschritt. |
@andig wollen wir, wo wir gerade dabei sind, nicht auch gleich
|
fd7e1e3
to
32550fe
Compare
Setup von evcc mit einer leeren setup.webm |
@andig ich hab gerade noch mal ausprobiert wie sich die Einstellungen von UI-konfigurierten Fahrzeugen an UI-konfigurierten Ladepunkten verhalten. Das sieht alles korrekt, bzw. wie erwartet aus. Die fahrzeugspezifischen Werte (bspw. Dementsprechend steht hier nur noch eine gute Lösung für den Demo-Modus aus. Hier tendiere ich gerade dazu das aktuelle Verhalten erst einmal zu belassen:
Ein explizites Anwählen des Demo Modus (bspw. durch CLI Param) würd ich erst dann machen, wenn wir Config UI nicht mehr hinter experimental Flag haben. |
Fix #12903
This PR deprecates the following yaml loadpoint settings:
TODO
evcc migrate
?) @andig @naltatisevcc migrate --reset
removes all ui configured entities (vehicles, meters, chargers, loadpoints)