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 chargemode #2079

Closed
wants to merge 2 commits into from
Closed

Conversation

LKuemmel
Copy link
Contributor

No description provided.

@MartinRinas
Copy link
Contributor

Wenn ich diesen PR richtig überflogen habe soll das SoC und Energielimit fürs Sofortladen entfernt werden, ist das richtig? Gibts einen Ersatz dafür oder fällt das komplett weg? Ich nutze das durchaus immer mal wieder.

@LKuemmel LKuemmel added this to the 2.1.7 milestone Dec 17, 2024
@LKuemmel LKuemmel marked this pull request as draft December 19, 2024 08:54
@LKuemmel LKuemmel removed this from the 2.1.7 milestone Dec 19, 2024
@hurraxdax
Copy link
Contributor

Anstatt das für Sofortladen zu entfernen (wird bei uns in der Familie auch regelmäßig genutzt) würde ich mir sehr wünschen diese Leiste mit den drei Buttons zum Umschalten/Abschalten der Begrenzung auch für PV-Laden einzbauen, mit der gleichen Funktionalität! Würde einerseits für eine konsistente Bedienung sorgen, andererseits möchte man ja vielleicht vielleicht auch beim PV-Laden mal nach Energiemenge begrenzen, bspw. wenn das BMW SoC-Modul mal wieder nicht geht.
Kenne mich leider mit dem vue-Zeug gar nicht aus, sonst hätte ich das selbst schon als PR eingereicht.

heidanei

@MartinRinas
Copy link
Contributor

Die Knöpfe kann ich bestimmt einbauen, das sollte nicht so arg schwer sein. Aber womöglich liege ich mit meiner Interpretation was hier passieren soll ja daneben. ICh bin ja durchaus ein Freund davon die Default-Experience möglichst einfach zu halten und gleichzeitig dem versierten Anwender mehr Optionen zu geben. Was immer leicht gesagt als getan ist, ich kann mir mit der UI fürs Limit beim Sofortladen gern mal daran versuchen.

@hurraxdax
Copy link
Contributor

Wäre cool, aber vielleicht erst mal sehen was Lena dazu sagt, nicht dass sich da die Änderungen beißen. Dieser PR scheint aktuell ja eh zurückgestellt zu sein. Ich bin ebenfalls ein Freund von möglichst einfach, drum finde ich auch die derzeitige Maske für PV-Laden im Standard-Theme auch viel zu kompliziert und überladen. Warum die Bedienung für die Begrenzung bei PV-Laden und Sofortladen komplett unterschiedlich ist erschließt sich mir nicht. Ich finde die Leiste mit den drei Buttons bei Sofortladen jedenfalls einfacher und übersichtlicher.

@LKuemmel
Copy link
Contributor Author

LKuemmel commented Jan 2, 2025

Wir überarbeiten das Konzept für die Lademodi-Einstellungen intern noch einmal. Daher schließe ich diesen PR. Bitte wartet mit Änderungen bis wir uns intern abgestimmt haben.

@LKuemmel LKuemmel closed this Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants