-
-
Notifications
You must be signed in to change notification settings - Fork 568
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
Implement introspectable actions #1588
Conversation
@rytilahti done processing feedback, could you have another look/merge? |
Codecov Report
@@ Coverage Diff @@
## master #1588 +/- ##
==========================================
+ Coverage 80.88% 80.91% +0.02%
==========================================
Files 156 156
Lines 15258 15276 +18
Branches 3283 3301 +18
==========================================
+ Hits 12342 12360 +18
Misses 2667 2667
Partials 249 249
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@rytilahti I think this can be merged. |
@rytilahti I added the missing actions, could you merge 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.
Tested to work as expected, thanks @starkillerOG! 👍
Adds the last missing introspectable properties (#1495), support for actions and implements them for RoborockVacuum.
The home assistant fork using this information is available at https://github.com/rytilahti/home-assistant/tree/xiaomi_miio/feat/entities_from_upstream
Split out from #1543