-
Notifications
You must be signed in to change notification settings - Fork 118
Add support for Smart Standing Fan 2 Pro #195
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
Conversation
I have committed some changes, and I came to the conclusion that - as you suggested - it made more sense to inherit from I initially thought that the attr names needed to be in line with what miot-spec would report in the JSON file, but that doesn't seem to be the case, so I aligned the names with the rest of the codebase. This potentially means that I can perhaps just use I have left some TODOs for which I might appreciate some help: 1 - The child lock command works correctly from HA, but the entity status keeps showing 2 - I noticed that I hope I am not giving you too many headaches with this, I appreciate the help |
Please add |
PS. You can ignore/change the attribute names of the miot specs. The IDs are the important keys. |
Did you fix the |
Yes, that's how I am currently testing everything. I am 100% sure the child lock property is correct. It's (7, 1), and when I activate it, I can manually see that it is correctly acrivated/deactivated, and
I can confirm that from off state,
Thank you!
Yes, it was one of those leftover properties that I didn't even need |
… Xiaomi Fan Lovelace Card
Did some more testing and I noticed that I had an issue with the delay timer (ZA5 was doing some minutes -> seconds conversion), so I took the time to test and fix it. It works correctly now. |
All changes done. So much of it is a reminder to not code the entire day, I dunno what the hell I was thinking with all those enum conversions lol Thanks for the help improving this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would be happy about a final test. If the implementation is working a expecting this is ready to get merged.
Please ping me if you are ready with testing and sure everything is working as expected. I would like to prepare a new release this evening. |
If with testing you mean "manually testing the device", I have tested it extensively since the last PR and it works correctly 👍 |
Thanks for your contribution! Feel free to prepare additional pull requests if you see room for improvements! |
Hi there, today I got myself a Smart Standing Fan 2 Pro, so I spent some time adding support to it.
I have tested all the supported functions and they all work correctly.
https://www.mi.com/global/product/xiaomi-smart-standing-fan-2-pro/