search

ioBroker / ioBroker.wolf

Connect WOLF heating, solar and domestic ventilation
MIT License
25 stars 16 forks source link

Made changes to get the adapter to work in my setup. + fix issue #58 + save function for dp149 #59

Closed Dark-Tower-Coder closed 2 years ago

Dark-Tower-Coder commented 2 years ago

As in the last PR (Made changes to get the adapter to work in my setup. #56) here are the same changes. In addition, issue #58 (Can't parse DP : 149 - length: 21) has been solved and for DP 149 it is now possible to save the values (the mode for the program).

Regarding my final approach for this PR: I cloned the original adapter and copied all the new changes from it to my repo, since there was Reales v1.3.4 in the meantime. I don't understand why the changes in the original code are missing in this PR. E.g. most changes in io-package.json.

I'll try to clean up the PR a bit. But I would appreciate feedback and a little help.

Apollon77 commented 2 years ago

@Dark-Tower-Coder why closed?

Apollon77 commented 2 years ago

@Dark-Tower-Coder ???

Dark-Tower-Coder commented 2 years ago

Sorry ich checks nicht.

Wenn ich einen PR mache gibt es keine Reaktion, kein Support und auch kein Feedback...

Ohne PR kann ich jetzt einfach programmieren ohne mir ein Kopf zu machen ob es zum originalen passt oder nicht.

Apollon77 commented 2 years ago

Bitte nicht falsch verstehen, ich versuche PRs schnell rein zu bekommen aber aktuell ist auch meine Zeit knapp und der js-controller 4 hatte iene höhere Priorität. Aber kein PR ist vergessen und ich komme dazu so schnell lich kann ... Kommentieren bzw Feedback ist immer schwierig wenn man ddas Gerät selbst nicht hat, dann ist das rein theoretischer Natur.

Also gern PR machen und ich priorisiere es (bin eh gerade am alles sammeln was ggf runtergfaallen ist), aber auch ich habe nur begrenzte Zeit und die muss ich einteilen

Dark-Tower-Coder commented 2 years ago

Alles gut. Nur mache ich mir ein Kopf ob alles passt und warte bis etwas zurück kommt. Also nicht unbedingt ob der Code an sich passt, sondern auch ob der PR soweit korrekt ist

Ich würde sagen ich gehe erstmal alles durch und passe an was noch für meine Hardware fehlt. Wenn ich dann fertig bin erstelle ich noch ein PR mit allen Änderungen.

Apollon77 commented 2 years ago

Ok, machen wir so ... sorry nochmal

Apollon77 commented 2 years ago

Wenn Du magst kann ich den PR trotzdem kommentieren code-wise

Dark-Tower-Coder commented 2 years ago

Wenn ich fertig bin und PR erstellt habe gerne. Ist ja alles nicht so dringend.

Mach dir kein Streß.

Apollon77 commented 2 years ago

@Dark-Tower-Coder whats status? I would prepare anything for an update too

Apollon77 commented 2 years ago

PS: I manually merged some of your changes, especially for fixing #58

Apollon77 commented 2 years ago

PS: I manually merged some of your changes, especially for fixing #58