Add useStrings config setting to make .toString() conversion optional

iRayanKhan / homebridge-tuya

A Homebridge plugin to control Tuya devices locally.

MIT License

380 stars 160 forks source link

Add useStrings config setting to make .toString() conversion optional #447

Closed scooterpsu closed 4 months ago

scooterpsu commented 5 months ago

Not sure exactly what makes this environmentally dependent, but my fan controllers don't speak string.

It's not the most elegant solution, so I'm open to alternatives. But it does work, and defaults to using .toString() if that's the more commonly used form.

05TEVE commented 4 months ago

Looks good. Will test this tonight and push out another beta. What brand of fan are you using? I am not entirely sure how consistent the implementations are across devices.

scooterpsu commented 4 months ago

I've got 3 of these installed in some ceiling fans: https://www.amazon.com/dp/B0BKJSQHKB

It was the only device I could find to control older fans where there's only one wire run to the wall switch. At least the only one that gave wall controls and not an RF remote.

05TEVE commented 4 months ago

Changes look good. Pushed a new beta out with them included. 3.1.1-beta.3