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

MK-2001 · 2024-01-15T14:54:45Z

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

* counting * Tower Object creation * Version match

* counting * Tower Object creation * Version match * Initialize array * init array * Debug * error handling * Bugfix Tower Number * Argh! * Debug * modified * . * Create Objects * . * words update * . * Fix for CellValues * updated text

removed local workflow

* counting * Tower Object creation * Version match * Initialize array * init array * Debug * error handling * Bugfix Tower Number * Argh! * Debug * modified * . * Create Objects * . * words update * . * Fix for CellValues * updated text * Added message when update

Enhanced Silly Debug; Issue in CellVoltageObjects;

MK-2001 · 2024-01-15T18:43:00Z

Closes #203

MK-2001 · 2024-01-16T12:55:37Z

@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 · 2024-01-16T16:08:44Z

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 · 2024-01-17T20:30:23Z

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 · 2024-01-17T20:42:06Z

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: ***@***.*** ***@***.***> >

Fix issue with christianh17#262

Fix issue with #262

MK-2001 and others added 10 commits January 15, 2024 11:21

Enable WF

87e5937

enable two tower

9f90543

pushed version

cf40adf

Onprem (#1)

c23a9b8

* counting * Tower Object creation * Version match

Onprem (#2)

456ac83

* counting * Tower Object creation * Version match * Initialize array * init array * Debug * error handling * Bugfix Tower Number * Argh! * Debug * modified * . * Create Objects * . * words update * . * Fix for CellValues * updated text

Delete .gitea/workflows/deploy.yml

0f2ec86

removed local workflow

Onprem (#3)

c1e743e

* counting * Tower Object creation * Version match * Initialize array * init array * Debug * error handling * Bugfix Tower Number * Argh! * Debug * modified * . * Create Objects * . * words update * . * Fix for CellValues * updated text * Added message when update

Onprem (#4)

f3b9a85

Enhanced Silly Debug; Issue in CellVoltageObjects;

Merge branch 'master' into master

06e6138

Handle Issue christianh17#203 (#5)

0799423

christianh17 merged commit c501826 into christianh17:master Jan 16, 2024

MK-2001 added a commit to MK-2001/ioBroker.bydhvs that referenced this pull request Jan 18, 2024

Update main.js

d677e6e

Fix issue with christianh17#262

MK-2001 mentioned this pull request Jan 18, 2024

Fix issue with #262 #267

Merged

christianh17 added a commit that referenced this pull request Feb 4, 2024

Merge pull request #267 from MK-2001/patch-1

9620ba6

Fix issue with #262

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

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

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

MK-2001 commented Jan 15, 2024 •

edited

Loading

MK-2001 commented Jan 15, 2024

MK-2001 commented Jan 16, 2024

christianh17 commented Jan 16, 2024

christianh17 commented Jan 17, 2024

christianh17 commented Jan 17, 2024 via email

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

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

Conversation

MK-2001 commented Jan 15, 2024 • edited Loading

MK-2001 commented Jan 15, 2024

MK-2001 commented Jan 16, 2024

christianh17 commented Jan 16, 2024

christianh17 commented Jan 17, 2024

christianh17 commented Jan 17, 2024 via email

MK-2001 commented Jan 15, 2024 •

edited

Loading