Skip to content
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

Merged
merged 4 commits into from
Mar 1, 2023
Merged

Simplify climater api #6397

merged 4 commits into from
Mar 1, 2023

Conversation

andig
Copy link
Member

@andig andig commented Feb 23, 2023

Fix #6385

TODO

  • update ui

@andig andig added the enhancement New feature or request label Feb 23, 2023
@@ -180,7 +180,7 @@ export default {
chargePower: Number,
chargedEnergy: Number,
// chargeDuration: Number,
climater: String,
climaterActive: Boolean,
Copy link
Member Author

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 :)

@andig andig requested a review from naltatis February 24, 2023 14:05
@andig
Copy link
Member Author

andig commented Feb 26, 2023

@naltatis scheint als würde UI hier gar nix tun (auch keine Nachricht)- oder übersehe ich da was?

@naltatis
Copy link
Member

naltatis commented Mar 1, 2023

Ja, ich glaube das Klimaisierungsthema ist bei einem Umbau damals rausgeflogen. Soll ich das in den Statusbereich (Text) wieder aufnehmen?

@andig
Copy link
Member Author

andig commented Mar 1, 2023

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.

@andig andig merged commit ae61e9e into master Mar 1, 2023
@andig andig deleted the feature/vehicle-climate branch March 1, 2023 10:28
@VolkerK62
Copy link
Contributor

gibt es hier etwas für die Doku zu tun?

@andig
Copy link
Member Author

andig commented Mar 1, 2023

Funktioniert weiter wie bisher- nur beim custom vehicle kann jetzt climater mit gepflegt werden und muss einen boolschen Wert liefern.

@VolkerK62
Copy link
Contributor

Ok. Kommt das hier https://docs.evcc.io/docs/devices/vehicles#manuell aus einem Template (vermutlich nein) oder muss das händisch angepasst werden?

@andig
Copy link
Member Author

andig commented Mar 1, 2023

Genau da, kein Template.

@andig
Copy link
Member Author

andig commented Mar 1, 2023

Klasse, vielen Dank 🙏

@VolkerK62
Copy link
Contributor

der PR wurde allerdings vom Github Check abgelehnt

@premultiply
Copy link
Member

Ich hab da glaub ich gestern was (anderes) kaputt gemacht. Muss ich gleich mal reparieren.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proposal: Vereinfachung der Klimaerkennung auf ein/aus als Basis für Unterstützung mittels Plugins
4 participants