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

Implement introspectable actions #1588

Merged
merged 9 commits into from
Nov 15, 2022
Merged

Conversation

starkillerOG
Copy link
Contributor

@starkillerOG starkillerOG commented Nov 9, 2022

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

miio/device.py Outdated Show resolved Hide resolved
miio/devicestatus.py Show resolved Hide resolved
@starkillerOG
Copy link
Contributor Author

@rytilahti done processing feedback, could you have another look/merge?

@codecov-commenter
Copy link

codecov-commenter commented Nov 9, 2022

Codecov Report

Merging #1588 (70cac3e) into master (10464c9) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            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              
Impacted Files Coverage Δ
miio/devicestatus.py 91.56% <100.00%> (+0.89%) ⬆️
miio/integrations/vacuum/roborock/vacuum.py 66.42% <100.00%> (+0.63%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@rytilahti rytilahti changed the title Implement actions Implement introspectable actions Nov 9, 2022
@starkillerOG
Copy link
Contributor Author

starkillerOG commented Nov 9, 2022

@rytilahti I think this can be merged.

@starkillerOG
Copy link
Contributor Author

@rytilahti I added the missing actions, could you merge this PR?

Copy link
Owner

@rytilahti rytilahti left a 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! 👍

image

@rytilahti rytilahti merged commit 0db7997 into rytilahti:master Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants