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 Xiaomi Airpurifier Pro V7 support #20093

Merged

Conversation

syssi
Copy link
Member

@syssi syssi commented Jan 14, 2019

Description:

It's a more complete implementation of #19653 of @quamilek.

airpurifier-pro-v7

Related issue (if applicable): syssi/xiaomi_airpurifier#31

Kudos to @quamilek @Mazon and @SebuZet for taking the device support forward.

Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#8169

@@ -577,6 +614,10 @@ def __init__(self, name, device, model, unique_id):
self._device_features = FEATURE_FLAGS_AIRPURIFIER_PRO
self._available_attributes = AVAILABLE_ATTRIBUTES_AIRPURIFIER_PRO
self._speed_list = OPERATION_MODES_AIRPURIFIER_PRO
elif self._model == MODEL_AIRPURIFIER_PRO_V7:
self._device_features = FEATURE_FLAGS_AIRPURIFIER_PRO_V7
self._available_attributes = AVAILABLE_ATTRIBUTES_AIRPURIFIER_PRO_V7

Choose a reason for hiding this comment

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

line too long (80 > 79 characters)

@frenck
Copy link
Member

frenck commented Jan 14, 2019

Docs need an update for this as well, adding docs-missing label.

Copy link
Member

@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!

@balloob balloob merged commit 8540478 into home-assistant:dev Jan 17, 2019
@ghost ghost removed the in progress label Jan 17, 2019
@balloob balloob mentioned this pull request Jan 23, 2019
alandtse pushed a commit to alandtse/home-assistant that referenced this pull request Feb 12, 2019
* Add Xiaomi Air Purifier Pro V7 support

* Reorder the model list

* Improve the list of supported attributes/properties

* Fix lint
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.

6 participants