From 30cba1e58d21a64e29716c254b9ba11fd59ae80a Mon Sep 17 00:00:00 2001 From: Claude review agent Date: Mon, 28 Sep 2026 15:06:12 +0200 Subject: [PATCH] UX-15 review: shorter setup banner (PR #147) The Home banner title was ten words; UX-08 asks for about six. Co-Authored-By: Claude Opus 5.5 --- goodwe_controller/DOCS.md | 11 +++++------ goodwe_controller/app/messages.py | 2 +- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/goodwe_controller/DOCS.md b/goodwe_controller/DOCS.md index 7493bdc..8a8712b 100644 --- a/goodwe_controller/DOCS.md +++ b/goodwe_controller/DOCS.md @@ -3244,9 +3244,8 @@ control is started, and whenever the stored settings name no inverter to control. A site that already ran before is never sent there. A new install names no inverter yet, so the add-on starts **in setup**: it -serves the page and sends nothing to any inverter. Home says "Setup: the -add-on controls nothing until setup is done", and there is no Start button. The -installer's steps: +serves the page and sends nothing to any inverter. Home says "Setup: nothing +is controlled yet", and there is no Start button. The installer's steps: 1. Install the add-on, turn on **Watchdog** on its Info tab, and start it. Leave the Configuration tab as it is. @@ -3276,9 +3275,9 @@ inverter stays at 0 W. If the banner is red and says a safety stop stays on, one of the red rows below is a stop that will still be there when shadow mode ends. That row says how to clear it. -In **setup** (a new install that names no inverter yet) the banner says the -add-on controls nothing until setup is done, and the one row says to finish -Setup. See *Setup, for a new install*. +In **setup** (a new install that names no inverter yet) the banner says +nothing is controlled yet, and the one row says to finish Setup. See *Setup, +for a new install*. | Row starts with | What it tells you | |---|---| diff --git a/goodwe_controller/app/messages.py b/goodwe_controller/app/messages.py index 4750030..32bb31d 100644 --- a/goodwe_controller/app/messages.py +++ b/goodwe_controller/app/messages.py @@ -385,7 +385,7 @@ def banner_stopped() -> str: def banner_setup_only() -> str: """UX-15: setup-only mode, where the add-on serves the page and commands nothing (app/setuponly.py).""" - return "Setup: the add-on controls nothing until setup is done" + return "Setup: nothing is controlled yet" def row_setup_only() -> str: