search

RWensveen / com.broadlink

Add support for Broadlink devices in Homey (by AthomBV)
GNU General Public License v3.0
10 stars 10 forks source link

Broadlink RM PRO PLUS and SP3, SP3S #4

Closed BibiMaxx closed 6 years ago

BibiMaxx commented 6 years ago

Hi Remco

When I added the rm Pro device for the first time in the app he recognized it now I want to add it again in the new version and i got the message shows" another device: RM2 PRO PLUS (10026)" As you know i tried the sp2 for the sp3 and then it's worked and now in the new version i got the the message " other device found: OEM Branded SPmini (10035)" Also i tried the SP3S and it's work but only the night switch is not there

Best regards

RWensveen commented 6 years ago

@BibiMaxx SP3S: it was not clear to me what the features of the SP3S were, compared to a SP2. I thought it had no nightlight. So I added the nightlight feature to the SP3S. SP2: small mistake on my side: it tries to verify a device id against the one returned by the detected device. I used the wrong device id in the SP2 driver :-( I have to use the RM3mini as substitute for most devices, as I do not have any. luckely you do :-)

BibiMaxx commented 6 years ago

Thanks for your answer. It's not a problem but how is it with the device "another device: RM2 PRO PLUS (10026)"

2018-07-06 21:45 GMT+02:00 RWensveen notifications@github.com:

@BibiMaxx https://github.com/BibiMaxx SP3S: it was not clear to me what the features of the SP3S were, compared to a SP2. I thought it had no nightlight. So I added the nightlight feature to the SP3S. SP2: small mistake on my side: it tries to verify a device id against the one returned by the detected device. I used the wrong device id in the SP2 driver :-( I have to use the RM3mini as substitute for most devices, as I do not have any. luckely you do :-)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/RWensveen/com.broadlink/issues/4#issuecomment-403129479, or mute the thread https://github.com/notifications/unsubscribe-auth/AmP4-xgJom6W4aqqdytGNYiEFStB7y2xks5uD75igaJpZM4VFRLi .

RWensveen commented 6 years ago

All solved in v1.3.2 :-)