Skip to content

Add surplus PV - Battery mode buttons to display - #2803

Merged
LKuemmel merged 7 commits into
openWB:masterfrom
Brett-S-OWB:flow-chart-battery-mode
Oct 31, 2025
Merged

Add surplus PV - Battery mode buttons to display#2803
LKuemmel merged 7 commits into
openWB:masterfrom
Brett-S-OWB:flow-chart-battery-mode

Conversation

@Brett-S-OWB

@Brett-S-OWB Brett-S-OWB commented Oct 1, 2025

Copy link
Copy Markdown
Contributor

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

benderl
benderl requested changes Oct 13, 2025
View reviewed changes

@benderl benderl left a comment

Copy link
Copy Markdown
Contributor

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.

Comment thread packages/modules/display_themes/cards/source/src/components/Battery/BatteryModeModal.vue
Comment thread packages/modules/display_themes/cards/source/src/components/Battery/BatteryModeModal.vue Outdated
@Brett-S-OWB

Brett-S-OWB commented Oct 13, 2025

Copy link
Copy Markdown
Contributor Author

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.

Schaltfläche / Anzeige zum Batteriebereich des Flussdiagramms hinzugefügt, um den Benutzer auf die Funktionalität aufmerksam zu machen.
Bildschirmfoto 2025-10-13 um 17 03 57
Bildschirmfoto 2025-10-13 um 17 05 07
Bildschirmfoto 2025-10-13 um 17 05 19

@Brett-S-OWB

Brett-S-OWB commented Oct 15, 2025

Copy link
Copy Markdown
Contributor Author

Anzeige des Batterielademodus in ein Badge geändert
Batterielademodi umbenannt
Bildschirmfoto 2025-10-14 um 16 45 59
Bildschirmfoto 2025-10-15 um 19 48 14

benderl
benderl approved these changes Oct 22, 2025
View reviewed changes

@benderl benderl left a comment

Copy link
Copy Markdown
Contributor

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.

@benderl benderl added this to the 2.1.9 milestone Oct 22, 2025
@benderl benderl added the enhancement New feature or request label Oct 22, 2025
@benderl benderl self-assigned this Oct 22, 2025
@Brett-S-OWB
Brett-S-OWB force-pushed the flow-chart-battery-mode branch from c80986e to 723b15b Compare October 22, 2025 08:27
@Brett-S-OWB

Brett-S-OWB commented Oct 22, 2025

Copy link
Copy Markdown
Contributor Author

Standardwert angegeben entfernt von modelValue wenn required: true definiert ist

@LKuemmel
LKuemmel merged commit 72b3bc6 into openWB:master Oct 31, 2025
5 checks passed
@Brett-S-OWB
Brett-S-OWB deleted the flow-chart-battery-mode branch December 2, 2025 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@benderl benderl benderl approved these changes

Assignees

@benderl benderl

Labels

enhancement New feature or request

Projects

None yet

Milestone

2.1.9

Development

Successfully merging this pull request may close these issues.

3 participants

@Brett-S-OWB @benderl @LKuemmel