Konfiguration für enecess EcoMain in den Geräteeinstellungen ergänzen - #1018
Konfiguration für enecess EcoMain in den Geräteeinstellungen ergänzen#1018enecess wants to merge 1 commit into
Conversation
|
Hallo zusammen, auch dieser UI-PR ist bewusst in zwei unabhängige Commits aufgeteilt:
Falls Legacy SmartHome nicht mehr aufgenommen werden soll, kann der zweite Commit vor dem Merge entfernt werden; wir reduzieren den PR dann gerne auf die Einstellungen der nativen Geräteintegration. Dies wird entsprechend mit dem Core-PR openWB/core#3681 abgestimmt. Das Seriennummernfeld in der Oberfläche gehört zur zusätzlichen Geräteidentitätsprüfung im Core. Sollte im Core-Review entschieden werden, auf diese Prüfung zu verzichten oder sie anders umzusetzen, passen wir die UI-Konfiguration selbstverständlich entsprechend an. |
|
Hallo, |
There was a problem hiding this comment.
Pull request overview
Ergänzt die UI-Konfiguration für enecess EcoMain sowohl in der Geräte-Konfiguration (neues UI) als auch in der Legacy SmartHome-Konponente, passend zur benötigten Core-Integration (openWB/core#3681, Bezug core#3657).
Changes:
- Neues Geräte-UI für EcoMain inkl. IP/Seriennummer sowie EVU- und Wechselrichter-Komponenten (inkl. Kanal-/Phasenlogik, Invert-Option).
- Legacy SmartHome: EcoMain als Messgerät inkl. neuer Felder (Seriennummer, Phasenanzahl, Invert, Quellen/Kanäle je Phase) + Sichtbarkeitslogik.
- Tests zur Absicherung der Legacy-SmartHome-Quellen und der Inverter-Konfigurationslogik.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/modules/legacy_smart_home.spec.js | Neue Tests, die EcoMain-Felder/Topics sowie Defaults/Dropdown-Implementierung in Legacy SmartHome absichern. |
| src/components/devices/enecess/ecomain/device.vue | Geräte-UI für EcoMain (IP/Hostname, Seriennummer, Mindest-Softwareversion Hinweis). |
| src/components/devices/enecess/ecomain/counter.vue | EVU-Zähler-Komponente (Hinweistext zu verfügbaren Werten). |
| src/components/devices/enecess/ecomain/inverter.vue | Wechselrichter-Konfiguration: Phasenanzahl, Invert, Kanal/Quelle/Phase je Kanal + Validierung/Unique-Checks. |
| src/components/devices/enecess/ecomain/inverter.spec.js | Unit-Tests für Inverter-UI-Logik (Dropdown-Optionen, Change-PhaseCount-Verhalten, Validierung). |
| public/modules/legacy_smart_home/topicsToSubscribe_smarthomeconfig.js | MQTT-Subscription um EcoMain-Konfig-Topics erweitert. |
| public/modules/legacy_smart_home/smarthomeconfig.php | EcoMain als Messgerät ergänzt inkl. UI-Elemente und Visibility-Handling für Phasenanzahl/Kanäle. |
| public/modules/legacy_smart_home/helperFunctions.js | Defaults-Reset für EcoMain-Kanal-Selects ergänzt. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Der Legacy-SmartHome-Commit wurde wie gewünscht entfernt. Dieser Pull Request enthält jetzt ausschließlich die Einstellungen für die native EcoMain-Geräteintegration. |
Zusammenfassung
Diese Änderung ergänzt die Einstellungsoberfläche für die EcoMain-Integration im openWB Core.
Gerätekonfiguration
Motivation
Die Oberfläche stellt alle Parameter bereit, die von der zugehörigen EcoMain-Integration im Core benötigt werden.
Bezug: openWB/core#3657
Validierung
Abhängigkeit
Dieser Pull Request gehört zur EcoMain-Core-Integration in openWB/core#3681.