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

Add support for zhimi.humidifier.ca4 (miot) #772

Merged
merged 11 commits into from
Jul 27, 2020
Merged

Conversation

Toxblh
Copy link
Contributor

@Toxblh Toxblh commented Jul 25, 2020

Added minimal functionality for work with zhimi.humidifier.ca4 and similar in future humidifiers.

Close one of device from the #627 issue

image

@coveralls
Copy link

coveralls commented Jul 25, 2020

Coverage Status

Coverage increased (+0.3%) to 74.03% when pulling 8de0c00 on Toxblh:master into eef6b36 on rytilahti:master.

@Toxblh Toxblh changed the title WIP: Add suport air humidifier via miot for zhimi.humidifier.ca4 Add suport air humidifier via miot for zhimi.humidifier.ca4 Jul 26, 2020
@Toxblh
Copy link
Contributor Author

Toxblh commented Jul 27, 2020

@rytilahti Hi, could you help me here, what is wrong with "Labeler / label" check?

@rytilahti
Copy link
Owner

Hey, first of all, thanks for the PR! I will do a code review soon.

Wrt the labeler (which simply adds labels to PRs), looks like it works on repositories it has write access (https://github.community/t/github-actions-are-severely-limited-on-prs/18179), rendering it pretty much useless for this project. I'm going to disable it for the time being, so no need to worry about that.

I'll do a code review soon

Copy link
Owner

@rytilahti rytilahti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks straightforward to me, just a couple of minor changes needed 👍

Could you also update the README file to include this device?

miio/airhumidifier_miot.py Outdated Show resolved Hide resolved
miio/airhumidifier_miot.py Outdated Show resolved Hide resolved
miio/airhumidifier_miot.py Outdated Show resolved Hide resolved
miio/airhumidifier_miot.py Outdated Show resolved Hide resolved
miio/airhumidifier_miot.py Show resolved Hide resolved
miio/airhumidifier_miot.py Show resolved Hide resolved
README.rst Show resolved Hide resolved
…ties. Added Logger for catch branch. Updated tests
@rytilahti rytilahti changed the title Add suport air humidifier via miot for zhimi.humidifier.ca4 Add support for zhimi.humidifier.ca4 (miot) Jul 27, 2020
Copy link
Owner

@rytilahti rytilahti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks again! 🎉

@rytilahti rytilahti merged commit b1dd7aa into rytilahti:master Jul 27, 2020
xvlady pushed a commit to xvlady/python-miio that referenced this pull request May 9, 2021
* Added minimal functionality for work air humidifier via miot for zhimi.humidifier.ca4

* Implemented all features of air humidifier miot

* Added tests for air humidifier miot

* Airhumidifier_miot added divided by 1.25 for water level for providing percentage of the level

* airhumidifier miot added and updated tests

* Added miio.airhumidifier_miot to docs

* AirHumidifier_Miot. Changed to consistent naming of methods for homeAssistant and other modules

* miio/airhumidifier_miot.py Remove the duplicate docstring.

Co-authored-by: Teemu R. <tpr@iki.fi>

* Update miio/airhumidifier_miot.py

Co-authored-by: Teemu R. <tpr@iki.fi>

* Added Xiaomi Mi Air Humidifier CA4 to Readme

* AirHumidifierMiot. State fault -> error. Consistent in desc of properties. Added Logger for catch branch. Updated tests

Co-authored-by: Teemu R. <tpr@iki.fi>
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

3 participants