-
-
Notifications
You must be signed in to change notification settings - Fork 574
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
Do not get battery status for mains powered devices #1131
Conversation
@rytilahti could you look at this and merge? |
Co-authored-by: Sebastian Muszynski <basti@linkt.de>
Co-authored-by: Sebastian Muszynski <basti@linkt.de>
@syssi any more suggestions? |
Codecov Report
@@ Coverage Diff @@
## master #1131 +/- ##
==========================================
- Coverage 74.87% 74.81% -0.06%
==========================================
Files 76 76
Lines 8999 9006 +7
Branches 767 769 +2
==========================================
Hits 6738 6738
- Misses 2077 2084 +7
Partials 184 184
Continue to review full report at Codecov.
|
I will merge this PR later the day. I want to give @rytilahti the chance to react. |
I think this can now be merged |
This schould fix the errors as reported here: #1121 (comment)
If the battery status is requested from a mains powered device like the
lumi.plug.mmeu01
it gives an error like:miio.exceptions.DeviceError: {'code': -4003, 'message': 'prop or action not exist'}