search

Apollon77 / node-mbus

Nodejs mbus module
MIT License
20 stars 15 forks source link

Update index.js #62

Closed ndom91 closed 4 years ago

ndom91 commented 4 years ago

Added check for result on line 129 as it would sometimes crash for me here due to result being undefined.

Apollon77 commented 4 years ago

Thank you for reporting the issue and also providing a PR!

I fixed it a bit different because this code part was flawn in general :-)

Apollon77 commented 4 years ago

see https://github.com/Apollon77/node-mbus/commit/83165fb59a53de69d7d0658d3d29e2c51beb09c7

Apollon77 commented 4 years ago

1.2.1 will be released at this moment.

ndom91 commented 4 years ago

Great thanks!