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

airhumidifer_(mj)jsq: Add use_time for better API compatibility #1179

Merged
merged 1 commit into from Nov 5, 2021

Conversation

rytilahti
Copy link
Owner

Not all humidifiers support use_time, this adds the properties to return None on those cases for better API compatibility among devices. The proper fix will be adding base classes to define the minimal common API, but that takes some time.

Spotted at home-assistant/core#59045

@rytilahti rytilahti merged commit 210e818 into master Nov 5, 2021
@rytilahti rytilahti deleted the humidifier/fix/add_noop_use_times branch November 5, 2021 14:42
@rytilahti rytilahti mentioned this pull request Nov 30, 2021
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.

None yet

1 participant