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

Reimplemented FanAccessory; Add Ceiling Fan Light support (fsd). #37

Merged
merged 1 commit into from
Nov 1, 2022

Conversation

0x5e
Copy link
Owner

@0x5e 0x5e commented Nov 1, 2022

tuya#300 and tuya#295 should fixed, please have a try.

@0x5e 0x5e merged commit fdb4f1e into develop Nov 1, 2022
@0x5e 0x5e deleted the feature/fan branch November 1, 2022 14:05
@0x5e
Copy link
Owner Author

0x5e commented Nov 1, 2022

Ceiling Fan Light is supported now in 1.6.0-beta.18. Since FanAccessory has beem reimplemented please tell me if it's not working well, thanks. @fercho2903 @iGageit

@iGageit
Copy link

iGageit commented Nov 1, 2022

Oh Wow! it works now! Thank you so much!
hey, I make videos on social media and would like to give you a shoutout for your incredible development skills. Any code projects you would like people to know about? :) @0x5e

@0x5e
Copy link
Owner Author

0x5e commented Nov 1, 2022

Oh Wow! it works now! Thank you so much!

hey, I make videos on social media and would like to give you a shoutout for your incredible development skills. Any code projects you would like people to know about? :) @0x5e

Thanks, actually my other projects are old or small which may not useful for others. Homebridge Tuya is the first one I would like to share maybe.
I just like open source community and because work at tuya before, so I have experience to do this to share with :)

@0x5e
Copy link
Owner Author

0x5e commented Nov 2, 2022

Also tuya#313

@ryanuy
Copy link

ryanuy commented Nov 6, 2022

@0x5e Im trying to added the fan light switch into homekit but only the fan is appearing. using 1.6.0-beta.25, is it removed on this build?

@0x5e
Copy link
Owner Author

0x5e commented Nov 6, 2022

@ryanuy have you used previous beta version before? Does it work?
Please following the readme troubleshooting section and upload the TuyaDeviceList.json, thanks.

@ryanuy
Copy link

ryanuy commented Nov 6, 2022

@ryanuy have you used previous beta version before? Does it work?
Please following the readme troubleshooting section and upload the TuyaDeviceList.json, thanks.

@0x5e , this is the first time I'm using your beta.. please see the file below. thank you.
TuyaDeviceList.json.txt

@0x5e
Copy link
Owner Author

0x5e commented Nov 6, 2022

From your file I can only see switch for fan switch and fan_speed for fan speed, I didn't see anything for lights. Are you sure your device support light? Can you take some photos on Tuya Platform -> Cloud Develop -> device debugging page of your devices? Take photos for the standard instruction set and standard status set.
@ryanuy

@0x5e 0x5e added this to the 1.6.0 milestone Nov 7, 2022
@ryanuy
Copy link

ryanuy commented Nov 7, 2022

From your file I can only see switch for fan switch and fan_speed for fan speed, I didn't see anything for lights. Are you sure your device support light? Can you take some photos on Tuya Platform -> Cloud Develop -> device debugging page of your devices? Take photos for the standard instruction set and standard status set. @ryanuy

@0x5e I check the Tuya IOT Platform and it looks like Tuya platform is not sending all the device info to the Tuya IOT Platform.

tuya_fan_light_switch

but on the tuya smart app it has both fan controls and light switch.

Tuya Smart App

@0x5e
Copy link
Owner Author

0x5e commented Nov 7, 2022

@ryanuy So, unfortunately your device have non-standard instrument set which are invisible from tuya platform, and we can't get it from OpenAPI too...
#44 (comment)

@ryanuy
Copy link

ryanuy commented Nov 7, 2022

I looks that way. thank you for you help.

@0x5e
Copy link
Owner Author

0x5e commented Nov 7, 2022

You are welcome

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