Skip to content

Fix leak sensors always showing Unknown until Wet#11313

Merged
pvizeli merged 1 commit into
home-assistant:devfrom
OverloadUT:fix-moisure-sensors-unknown-state
Dec 27, 2017
Merged

Fix leak sensors always showing Unknown until Wet#11313
pvizeli merged 1 commit into
home-assistant:devfrom
OverloadUT:fix-moisure-sensors-unknown-state

Conversation

@OverloadUT
Copy link
Copy Markdown
Contributor

Description:

Bugfix for my changes in 0.60.

Insteon Leak sensors were using the "wet" node as a negative node, which prevented them from ever getting a Dry status unless the user pressed the button on the hardware after every Hass reboot.

This change ignores the Wet node, as it is just always the exact inverse of the Dry node. We don't need to watch both.

Checklist:

If the code communicates with devices, web services, or third-party tools:

  • Local tests with tox run successfully. Your PR cannot be merged unless tests pass

Leak sensors were using the "wet" node as a negative node, which prevented them from ever gettng a Dry status unless the user pressed the button on the hardware after every Hass reboot.

This change ignores the Wet node, as it is just always the exact inverse of the Dry node. We don't need to watch both.
@pvizeli pvizeli added this to the 0.60.1 milestone Dec 27, 2017
@pvizeli pvizeli merged commit aa8db78 into home-assistant:dev Dec 27, 2017
@OverloadUT OverloadUT deleted the fix-moisure-sensors-unknown-state branch December 28, 2017 20:12
balloob pushed a commit that referenced this pull request Jan 5, 2018
Leak sensors were using the "wet" node as a negative node, which prevented them from ever gettng a Dry status unless the user pressed the button on the hardware after every Hass reboot.

This change ignores the Wet node, as it is just always the exact inverse of the Dry node. We don't need to watch both.
@balloob balloob mentioned this pull request Jan 5, 2018
@home-assistant home-assistant locked and limited conversation to collaborators May 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants