-
Notifications
You must be signed in to change notification settings - Fork 107
Add surplus PV - Battery mode buttons to display #2803
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
Conversation
benderl
left a comment
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.
Die Verknküpfung mit der Batterie im Flow-Chart finde ich nicht gerade intuitiv, weil die aktuelle Einstellung des Modus dort nicht angezeigt wird. Ich würde nicht auf die Idee kommen, dass sich dahinter die Einstellungen der Batterie befinden. Der aktuelle Modus sollte irgendwie angezeigt werden, damit es intuitiver wird.
packages/modules/display_themes/cards/source/src/components/Battery/BatteryModeModal.vue
Show resolved
Hide resolved
packages/modules/display_themes/cards/source/src/components/Battery/BatteryModeModal.vue
Outdated
Show resolved
Hide resolved
benderl
left a comment
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.
Kannst Du noch die Warnungen beheben?
Das Problem ist, dass modelValue als required: true definiert ist und trotzdem ein Standardwert angegeben ist. Letzterer wird demnach nie verwendet und kann entfallen.
c80986e to
723b15b
Compare
|
Standardwert angegeben entfernt von modelValue wenn required: true definiert ist |





Display Cards-Theme: im Energiefluss beim klicken auf Speicher den Speicher-Vorrang (EV, Speicher, Mindest-SoC) auswählen