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

Fix ZHA battery when readings produce an unknown value #23854

Merged
merged 3 commits into from
May 14, 2019

Conversation

dmulcahey
Copy link
Contributor

@dmulcahey dmulcahey commented May 14, 2019

Description:

Fixes an issue with ZHA battery readings when the device returns 0 or 255 for the battery reading. Fixes: #23827

Depends on: #23855

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • I have followed the [development checklist][dev-checklist]

@dmulcahey dmulcahey added this to the 0.93 milestone May 14, 2019
@dmulcahey dmulcahey requested a review from Adminiuga May 14, 2019 00:59
@homeassistant homeassistant added cla-signed small-pr PRs with less than 30 lines. labels May 14, 2019
@ghost ghost assigned Adminiuga and dmulcahey May 14, 2019
@ghost
Copy link

ghost commented May 14, 2019

Hey there @Adminiuga, mind taking a look at this pull request as its been labeled with a integration (zha) you are listed as a codeowner for? Thanks!

This is a automatic comment generated by codeowners-mention to help ensure issues and pull requests are seen by the right people.

@balloob balloob merged commit 9b21774 into home-assistant:dev May 14, 2019
balloob pushed a commit that referenced this pull request May 14, 2019
* check for unknown readings

* only publish valid readings

* remove unused constant
@balloob balloob mentioned this pull request May 14, 2019
@dmulcahey dmulcahey deleted the dm/zha-battery branch July 18, 2019 10:54
alandtse pushed a commit to alandtse/home-assistant that referenced this pull request Oct 12, 2019
…t#23854)

* check for unknown readings

* only publish valid readings

* remove unused constant
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.

ZHA - "TypeError: unsupported operand type(s) for /: 'str' and 'int'"
4 participants