node-red-contrib-tradfri icon indicating copy to clipboard operation
node-red-contrib-tradfri copied to clipboard

Published 20 hours ago • verified publisher icon nidayand

Max Brightness

Open gjbianco opened this issue 5 years ago • 0 comments

Currently, trying to send a brightness of 255 just causes the command to be ignored. It seems like it is known that 254 is the actual max:

https://github.com/nidayand/node-red-contrib-tradfri/issues/19#issuecomment-381088494

Either the documentation should be updated to reflect this or, better yet, clamp the brightness value being passed in. e.g. sending a brightness > 254 causes 254 to be used.

I should be able to open a PR for this hopefully sometime this week, if that would be helpful.

Edit: looks like the docs were changed last year but the change was never released?

gjbianco avatar Jul 10 '19 14:07 gjbianco