Hinzufügen des Victron 3P75CT Moduls#3143
Open
AlexHuebi wants to merge 9 commits intoopenWB:masterfrom
Open
Conversation
Contributor
|
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. |
Author
|
Danke für die Info, habe soeben dort den PR erstellt. |
Contributor
|
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 |
ndrsnhs
approved these changes
Feb 23, 2026
LKuemmel
requested changes
Feb 24, 2026
Contributor
LKuemmel
left a comment
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
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.