search

christianh17 / ioBroker.bydhvs

BYD HVS Battery Adapter for ioBroker
MIT License
18 stars 10 forks source link

Enable the analysis of two towers. [BREAKING] #262

Closed MK-2001 closed 5 months ago

MK-2001 commented 5 months ago

introducing the structure bydhvs.0.Diagnosis.Tower_1. [...] and bydhvs.0.CellDetails.Tower_1. [...]

Closes #198

If there is something missing, let it me know to merge this PR into 1.5.1

MK-2001 commented 5 months ago

Closes #203

MK-2001 commented 5 months ago

@christianh17 Hallo Christian, ich habe eine Erweiterung deines gelungenen Adapters gebaut uns diese bei mir nun einen Tag erfolgreich getestet. Könntest du bitte den PR mergen? Gruß Marcus

christianh17 commented 5 months ago

Hallo Marcus! Habe vorhin den Merge gemacht. Danke!!! Das ist etwas wo ich schon ewig ran wollte aber keine Zeit hatte. Ich installiere heute Abend auf meinem ioBroker den Adapter aus dem git, wenn alles klappt und er auch bei mir funktioniert stecke ich die neue Version ins unstable repository. Viele Grüße Christian

christianh17 commented 5 months ago

Hallo Marcus! Leider läuft der Adapter bei mir in der neuen Version nicht, ` bydhvs.0 2024-01-17 21:10:42.259 error Cannot set properties of undefined (setting 'hvsMaxmVolt')

bydhvs.0 2024-01-17 21:10:42.259 error TypeError: Cannot set properties of undefined (setting 'hvsMaxmVolt') at decodePacket5 (/opt/iobroker/node_modules/iobroker.bydhvs/main.js:557:46) at Socket. (/opt/iobroker/node_modules/iobroker.bydhvs/main.js:834:13) at Socket.emit (node:events:517:28) at addChunk (node:internal/streams/readable:368:12) at readableAddChunk (node:internal/streams/readable:341:9) at Readable.push (node:internal/streams/readable:278:10) at TCP.onStreamRead (node:internal/stream_base_commons:190:23)

bydhvs.0 2024-01-17 21:10:42.258 error uncaught exception: Cannot set properties of undefined (setting 'hvsMaxmVolt')`

Bevor ich versuche mich in die Änderungen einzudenken: Magst Du Dir das einmal ansehen? Ich habe den PR gemergt und aus der URL installiert.

Danke!

Viele Grüße Christian

christianh17 commented 5 months ago

Hallo Marcus!

  Nur zur Sicherheit, bekommst Du eine Info über den Kommentar den ich bei Github geschrieben habe? Der Adapter läuft leider bei mir nicht ☹.

Viele Grüße

Christian

  Von: Marcus K @.> Gesendet: Dienstag, 16. Januar 2024 13:56 An: christianh17/ioBroker.bydhvs @.> Cc: christianh17 @.>; Mention @.> Betreff: Re: [christianh17/ioBroker.bydhvs] Enable the analysis of two towers. [BREAKING] (PR #262)

  @christianh17 Hallo Christian, ich habe eine Erweiterung deines gelungenen Adapters gebaut uns diese bei mir nun einen Tag erfolgreich getestet. Könntest du bitte den PR mergen? Gruß Marcus

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @. @.> >