Hinzufügen des Victron 3P75CT Moduls#3143
Conversation
|
Die Konfig im Frontend wird über dieses Repo https://github.com/openWB/openwb-ui-settings gesteuert. Hier genauso deinen Fork erstellen und dann ein Modul für den neuen Zähler erstellen und einen PR stellen und hier mit verlinken, dann kanns gemeinsam reviewed und gemerged werden. |
|
Danke für die Info, habe soeben dort den PR erstellt. |
|
Der PR sieht sehr gut aus. Hast du noch den Link zum PR für die Änderungen im UI? Edit: gefunden openWB/openwb-ui-settings#903 |
LKuemmel
left a comment
There was a problem hiding this comment.
Bitte noch die Formatierung nach Flake8 anpasssen. Für die pytests wird wahrscheinlich noch die Ergänzung von ModbusUDP hier
core/packages/modules/conftest.py
Line 25 in 5615993
|
@LKuemmel openWB/openwb-ui-settings#903 würde hierfür im UI noch fehlen :) |
Hallo.
Da ich mich Aufgrund eines Problems wieder mit der Codebasis beschäftigen musste, habe ich auch verstanden wie die Module im Core zusammenarbeiten und konnte daher ein "funktionierendes" Modul implementieren welches nicht die bestehenden Funktionen zerstört.
Teile des Patches wurden von #2811 übernommen.
Dieser PR beinhaltet auch #3142 - Modbus UDP Funktionalität.
Im Backend sollte nun soweit alles Funktionieren, sieht im Webinterface wie folgt aus:

Bei der Gerätekonfigurationsseite muss noch etwas angepasst werden:

Falls ich noch etwas testen soll, bitte Benachrichtigen. Aber dz. sieht alles wie erwartet funktional aus.
Aber ich konnte leider nicht herausfinden, wie ich die Konfigurationsseite anpasse. Für mich als Laie im Webdev Bereich sieht es so aus wie wenn dieser Generiert wird.