-
Notifications
You must be signed in to change notification settings - Fork 145
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
CB-8929 Use PowerManager to get battery state on Win 10 #42
Conversation
}, { keepCallback: true }); | ||
} | ||
|
||
PowerManager.onremainingchargepercentchanged = reportStatus; |
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.
Would not it be better to use add/remove event listener here instead to avoid overriding a listener which might be set in an app code?
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.
Should we also enable the tests for Windows 10?
Tested and it works good on all targets (Win 10 desktop, Win 10 Mobile, Windows Phone 8.1). |
@daserge, addressed your notes |
@vladimir-kotikov LGTM, thanks! |
Platforms affected
Windows (specifically Windows 10)
What does this PR do?
Adds support for Windows 10
What testing has been done on this change?
Manual testing
Checklist