-
-
Notifications
You must be signed in to change notification settings - Fork 725
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
Simplify climater api #6397
Simplify climater api #6397
Conversation
@@ -180,7 +180,7 @@ export default { | |||
chargePower: Number, | |||
chargedEnergy: Number, | |||
// chargeDuration: Number, | |||
climater: String, | |||
climaterActive: Boolean, |
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.
@naltatis ich hab nicht rausgefunden, wo das eigentlich verwendet wird? Bei irgendeinem Refactoring rausgeflogen? Dann wären wir hier schon fertig :)
@naltatis scheint als würde UI hier gar nix tun (auch keine Nachricht)- oder übersehe ich da was? |
Ja, ich glaube das Klimaisierungsthema ist bei einem Umbau damals rausgeflogen. Soll ich das in den Statusbereich (Text) wieder aufnehmen? |
Das wäre gut, sonst wundert man sich immer, warum eigentlich geladen wird. In jedem Fall kann dann der PR hier schonmal rein- Ursache/Wirkung sind jetzt klar. |
gibt es hier etwas für die Doku zu tun? |
Funktioniert weiter wie bisher- nur beim custom vehicle kann jetzt |
Ok. Kommt das hier https://docs.evcc.io/docs/devices/vehicles#manuell aus einem Template (vermutlich nein) oder muss das händisch angepasst werden? |
Genau da, kein Template. |
Klasse, vielen Dank 🙏 |
der PR wurde allerdings vom Github Check abgelehnt |
Ich hab da glaub ich gestern was (anderes) kaputt gemacht. Muss ich gleich mal reparieren. |
Fix #6385
TODO