search

git-kick / ioBroker.e3dc-rscp

Manage E3/DC power station based on RSCP
GNU General Public License v3.0
26 stars 10 forks source link

SmartCharging features implemented #187

Closed ka-vaNu closed 1 year ago

ka-vaNu commented 1 year ago
ka-vaNu commented 1 year ago

Gerne! Für mich war das auch nochmal gut die Doku der Projekte zu haben. Irgenwo war dazu auch mal ein Issue...

Übernimm du das gerne mit den Setter. Ich komm mit dem Code rund um mapChangedIdToSetTags nicht so richtig klar. Wahrscheinlich kommt daher die unnötige Komplexität und die Änderungen bei den Tags. Ich habe an der Stelle die Automatismen noch nicht ganz durchschaut...

ka-vaNu commented 1 year ago

Ich denke der einfachste Weg ist, dass du einen neuen Branch erstellst und ich den Pull-Request gegen diesen Branch neu erstellt. Dann kannst du auf dem Branch alles fertig machen und ihn anschließend mit deinem Master mergen...

git-kick commented 1 year ago

Gerne! Für mich war das auch nochmal gut die Doku der Projekte zu haben. Irgenwo war dazu auch mal ein Issue...

Übernimm du das gerne mit den Setter. Ich komm mit dem Code rund um mapChangedIdToSetTags nicht so richtig klar. Wahrscheinlich kommt daher die unnötige Komplexität und die Änderungen bei den Tags. Ich habe an der Stelle die Automatismen noch nicht ganz durchschaut...

OK, dann merge ich demnächst den PR und arbeite dann weiter.

Der Mechanismus mit mapChangedIdToSetTags ist recht tricky (also: unschön), war aber bisher das Beste, was mir einfiel. Ich versuche, das im Kommentar noch etwas besser zu erklären. Danke, dass du dich so intensiv mit meinem Code auseinandersetzt.