search

AdyRock / com.misol

GNU General Public License v3.0
0 stars 1 forks source link

Can only add 7 soil sensors on a gateway #1

Closed skarsboe closed 2 years ago

skarsboe commented 3 years ago

I was unable to add my 8th soil sensor to my gateway even though I was receiving data in the log. I had a look at the code for onPairListDevices() in \com.misol-master\drivers\soil_moisture\driver.js and found an issue with the for loop.

The for loop for (var i = 1; i < 8; i++) needs to be increased to <9 or <=8 in order include the last sensor on the gateway.

I have tested it on my Homey, and I can now add all 8 soil sensors.

AdyRock commented 3 years ago

Ok, thanks for reporting it and providing detailed information. I will fix it as soon as I am back home.