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.

Sign up for GitHub

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

Jump to bottom

fix: update sensor.py for WASHING_MACHINE type devices #64

Merged
merged 1 commit into from
Mar 25, 2023
Merged

fix: update sensor.py for WASHING_MACHINE type devices #64

merged 1 commit into from
Mar 25, 2023

Conversation

ndom91
Copy link
Contributor

@ndom91 ndom91 commented Mar 25, 2023

Fix sensor.py for WASHING_MACHINE type

  • Remove extra time variable reference out of scope
  • Conditionally assign onOffStatus
  • Update HonBaseCurrentWaterUsed from state_class: 'MEASUREMENT' to TOTAL to appease HA statistics validation

@ndom91 ndom91 changed the title fix: update sensor for WASHING_MACHINE type fix: update sensor.py for WASHING_MACHINE type Mar 25, 2023
@ndom91 ndom91 changed the title fix: update sensor.py for WASHING_MACHINE type fix: update sensor.py for WASHING_MACHINE type devices Mar 25, 2023
@ndom91 ndom91 mentioned this pull request Mar 25, 2023
Closed
@gvigroux gvigroux merged commit 7341aea into gvigroux:master Mar 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ndom91 @gvigroux