Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update fanv2_accessory.js #255

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

onichan88
Copy link

My ceiling fan has the var named 'switch_led', but I left also the 'light' value for other people.
It works, I forked this in my repo and with this change and adding case 'fsd' in the index.js, it worked.

My ceiling fan has the var named switch_led, but I left also the light value for other people.
It works, I forked this in my repo and with this change and adding case 'fsd' in the index.js, it worked.
@dipakparmar
Copy link

@onichan88 added your fix to the fork of this project that I'm thinking of starting maintaining: dipakparmar/tuya-homebridge@039b0af

I will publish the new version to npm in a day or two after merging other PRS!

Thanks for your contribution! 😄

@aljorhythm
Copy link

Consider using an array for easier reading

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants