Skip to content

Conversation

LKuemmel
Copy link
Contributor

@LKuemmel LKuemmel commented Oct 7, 2022

No description provided.

@LKuemmel LKuemmel requested a review from benderl October 7, 2022 13:15
Copy link
Contributor

@benderl benderl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Siehe inline Kommentare.

@@ -5,7 +5,8 @@
<span class="small">(Modul: {{ $options.name }})</span>
</openwb-base-heading>
<openwb-base-alert subtype="info">
Diese Komponente erfordert keine Einstellungen.
Ausgelesen wird Register 19026 auf Port 502. ModbusTCP muss im
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das stimmt nicht ganz, da auch noch weitere Register ausgelesen werden. Ich würde es komplett weglassen. Einfach nur ein Hinweis auf die Modbus-Einstellungen:
"ModbusTCP muss im Janitza auf Port 502 aktiv und die ID auf "1" eingestellt sein."
Später kann man sicher noch überlegen, ob Port und ID nicht als Option konfigurierbar gemacht werden spollten.

@@ -4,6 +4,13 @@
Einstellungen für SMA-HM/EM Wechselrichter
<span class="small">(Modul: {{ $options.name }})</span>
</openwb-base-heading>
<openwb-base-alert subtype="info">
Dies ist nur die richtige Komponente, wenn ein extra EnergyMeter nur
für die PV vorhanden ist. Im Normalfall ist muss ein Gerät "SMA
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
für die PV vorhanden ist. Im Normalfall ist muss ein Gerät "SMA
für die PV vorhanden ist. Im Normalfall muss ein Gerät "SMA

für die PV vorhanden ist. Im Normalfall ist muss ein Gerät "SMA
Sunny Boy" mit der entsprechenden Wechselrichter-Komponente angelegt
werden. Ist nur ein HomeManager vorhanden ist dieses Modul nicht
richtig.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Info ergänzen, welche(s) Modul(e) genutzt werden können.


<script>
export default {
name: "DeviceSunnyBoyBat",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Der Name des Moduls wird bereits in bat.vue genutzt und sollte eindeutig sein. Selbes gilt für die Klasse des ersten DIV im Template.

Suggested change
name: "DeviceSunnyBoyBat",
name: "DeviceSunnyBoyBatSmartEnergy",

updateConfiguration($event, 'configuration.port')
"
>
<template #help> Standardmäßig Port 502 oder 1502. </template>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<template #help> Standardmäßig Port 502 oder 1502. </template>
<template #help>
Je nach Produktionsdatum und Softwareversion ist
502 oder 1502 ab Werk eingestellt. Bitte im
Wechselrichter überprüfen.
</template>

@@ -4,6 +4,11 @@
Einstellungen für SonnenBatterie
<span class="small">(Modul: {{ $options.name }})</span>
</openwb-base-heading>
<openwb-base-alert subtype="info">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Der Hinweis gehört in die Zähler Komponente.

@LKuemmel LKuemmel merged commit 2a6b014 into openWB:main Oct 10, 2022
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.

2 participants