-
-
Notifications
You must be signed in to change notification settings - Fork 666
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
Refactor session settings and apis (BC) #10335
Conversation
2cb07ef
to
9aca861
Compare
93b456e
to
3c994c5
Compare
22d000c
to
5b2fbbe
Compare
490d0c7
to
88a42b0
Compare
@naltatis was ist hier noch offen? |
@andig kannst du |
@andig Beim Fahrzeugwechsel scheint es noch Probleme zu geben. Der Fahrzeugwechsel über die UI funktioniert. Allerdings kann aktuell ein Fahrzeug an mehreren LPs gleichzeitig angemeldet sein. Vorher war das anders. Da wurde das Fahrzeug vom Loadpoint entfernt, falls es schon woanders verbunden war. Beim Publishing der Reichweite (vehicleRange) scheint es auch ein Problem zu geben. Initial ist alles korrekt. Wechselt man über die UI das Fahrzeug so verschwindet die Reichweite obwohl das ausgewählte Fahrzeug eine Reichweite hat. |
Inkl. Reichweitenschätzung, sofern möglich. Siehe #10650 |
Als |
Verstehe ich nicht. Wieso hier nicht |
Kannst du mir sagen wo wir das schon als name bezeichnet haben? Ich steh grad auf dem Schlauch. |
@andig all clear |
@naltatis @GrimmiMeloni lohnt es für
noch eine Migration zu schreiben, die genau 1x läuft? |
Was ist denn für den normal Anwender die Auswirkung wenn wir diese Migration nicht machen? |
I.W. Batteriesettings sind neu einzugeben. Kein grosses Ding. |
Dann würde ich das nicht migrieren. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Krasser Change. Irgendwas haben wir bestimmt übersehen. Aber durch Draufgucken find ich gerade nichts mehr. Lets try.
So be it :) Morgen werden wir in einer anderen Welt aufwachen :) |
Fix #10330, fix #9235, fix #9245, fix #9641, fix #10696, fix #10650
Functional changes:
TODO
limitSoc
fromActionConfig
and add to vehicle APIsite.
prefix from settings (BC break)limitSoc
to Arrival UI @naltatisresetondisconnect
@naltatisvehicleName
at loadpointBC Breaks
Global:
uri
removed, usenetwork
insteadSite:
site.
prefix from db)pvs
bypv
batteries
bybattery
vehicles
- no longer neededprioritySoc
(adjustable in ui)bufferSoc
(adjustable in ui)bufferStartSoc
(adjustable in ui)Loadpoint:
mode
is now persistedresetondisconnect
removed- ifmode
is configured it is used when vehicle is disconnectedsoc.min
(adjustable in plan ui -> Arrival)soc.target
(adjustable in by plan ui -> Arrival)