Fix many bugs

.github/workflows/hacs_check.yml

@@ -0,0 +1,17 @@
+name: HACS Action
+
+on:
+  push:
+  pull_request:
+  schedule:
+    - cron: "0 0 * * *"
+
+jobs:
+  hacs:
+    name: HACS Action
+    runs-on: "ubuntu-latest"
+    steps:
+      - name: HACS Action
+        uses: "hacs/action@main"
+        with:
+          category: "integration"

.github/workflows/validate.yml

@@ -0,0 +1,14 @@
+name: Validate with hassfest
+
+on:
+  push:
+  pull_request:
+  schedule:
+    - cron:  '0 0 * * *'
+
+jobs:
+  validate:
+    runs-on: "ubuntu-latest"
+    steps:
+        - uses: "actions/checkout@v3"
+        - uses: "home-assistant/actions/hassfest@master"

README.md

@@ -1,10 +1,13 @@
-# hOn
-Home Assistant component supporting hOn cloud.
+# Haier hOn
+[![hacs_badge](https://img.shields.io/badge/HACS-Custom-41BDF5.svg)](https://github.com/hacs/integration)
+
+Home Assistant component supporting devices of Haier's mobile app **hOn**.
 
 ## Installation
-1. Installing via HACS
+#### Installing via HACS
+1. You need to have installed [HACS](https://hacs.xyz/)
 2. Go to HACS->Integrations
-3. Add this repo into your HACS custom repositories
+3. Add this repo (`https://github.com/Andre0512/hon.git`) into your HACS custom repositories
 4. Search for Haier hOn and Download it
 5. Restart your HomeAssistant
 6. Go to Settings->Devices & Services
@@ -13,6 +16,51 @@ Home Assistant component supporting hOn cloud.
 9. Search for Haier hOn 
 10. Type your username used in the hOn App and hit submit
 
+## Contribute
+Any kind of contribution is welcome!
+#### Add appliances or additional attributes
+1. Install [pyhOn](https://github.com/Andre0512/pyhOn)
+   ```commandline
+    $ pip install pyhOn
+    ```
+2. Use the commandline tool to read out all appliance data from your account
+    ```commandline
+    $ pyhOn
+    User for hOn account: user.name@example.com
+    Password for hOn account: ********
+    ========== WM - Washing Machine ==========
+    commands:
+      pauseProgram: pauseProgram command
+      resumeProgram: resumeProgram command
+      startProgram: startProgram command
+      stopProgram: stopProgram command
+    data:
+      actualWeight: 0
+      airWashTempLevel: 0
+      airWashTime: 0
+      antiAllergyStatus: 0
+      ...
+    ```
+3. Fork this repository and clone it to your local machine
+4. Add the keys of the attributes you'd like to have as `EntityDescription` into this Repository  
+   _Example: Add pause button_
+    ```python
+    BUTTONS: dict[str, tuple[ButtonEntityDescription, ...]] = {
+        "WM": (                        # WM is the applianceTypeName
+            ButtonEntityDescription(
+                key="pauseProgram",    # key from pyhOn
+                name="Pause Program",  # name in home assistant
+                icon="mdi:pause",      # icon in home assistant
+                ...
+            ),
+        ...
+    ```
+5. Create a [pull request](https://github.com/Andre0512/hon/pulls)
+
+#### Tips and Tricks
+- If you want to have some states humanreadable, have a look at the `translation_key` parameter of the `EntityDescription`
+- If you need to implement some more logic, create a pull request to the underlying library. There we collect special requirements in the `appliances` directory.
+
 ## Supported Appliances
 - Washing Machine
 

custom_components/hOn/manifest.json

@@ -1,10 +0,0 @@
-{
-  "domain": "hon",
-  "name": "hOn",
-  "config_flow": true,
-  "version": "0.1.0",
-  "codeowners": ["@Andre0512"],
-  "iot_class": "cloud_polling",
-  "requirements": ["pyhOn==0.2.4"],
-  "documentation": "https://github.com/Andre0512/hOn/"
-}

custom_components/hon/binary_sensor.py

@@ -26,16 +26,16 @@ class HonBinarySensorEntityDescription(HonBinarySensorEntityDescriptionMixin, Bi
 BINARY_SENSORS: dict[str, tuple[HonBinarySensorEntityDescription, ...]] = {
     "WM": (
         HonBinarySensorEntityDescription(
-            key="lastConnEvent.category",
+            key="attributes.lastConnEvent.category",
             name="Connection",
             device_class=BinarySensorDeviceClass.CONNECTIVITY,
             on_value="CONNECTED",
         ),
         HonBinarySensorEntityDescription(
             key="doorLockStatus",
-            name="Door Locked",
+            name="Door",
             device_class=BinarySensorDeviceClass.DOOR,
-            on_value="1",
+            on_value="0",
         ),
     )
 }
@@ -53,9 +53,9 @@ async def async_setup_entry(hass, entry: ConfigEntry, async_add_entities) -> Non
             hass.data[DOMAIN]["coordinators"][device.mac_address] = coordinator
         await coordinator.async_config_entry_first_refresh()
 
-        if descriptions := BINARY_SENSORS.get(device.appliance_type_name):
+        if descriptions := BINARY_SENSORS.get(device.appliance_type):
             for description in descriptions:
-                if not device.data.get(description.key):
+                if not device.get(description.key):
                     _LOGGER.info("Can't setup %s", description.key)
                     continue
                 appliances.extend([
@@ -78,9 +78,9 @@ class HonBinarySensorEntity(HonEntity, BinarySensorEntity):
 
     @property
     def is_on(self) -> bool:
-        return self._device.data.get(self.entity_description.key, "") == self.entity_description.on_value
+        return self._device.get(self.entity_description.key, "") == self.entity_description.on_value
 
     @callback
     def _handle_coordinator_update(self):
-        self._attr_native_value = self._device.data.get(self.entity_description.key, "")
+        self._attr_native_value = self._device.get(self.entity_description.key, "") == self.entity_description.on_value
         self.async_write_ha_state()

custom_components/hon/button.py

@@ -9,16 +9,16 @@ from .hon import HonCoordinator, HonEntity
 
 BUTTONS: dict[str, tuple[ButtonEntityDescription, ...]] = {
     "WM": (
-        ButtonEntityDescription(
-            key="pauseProgram",
-            name="Pause Program",
-            icon="mdi:pause",
-        ),
-        ButtonEntityDescription(
-            key="resumeProgram",
-            name="Resume Program",
-            icon="mdi:play-pause",
-        ),
+        # ButtonEntityDescription(
+        #     key="pauseProgram",
+        #     name="Pause Program",
+        #     icon="mdi:pause",
+        # ),
+        # ButtonEntityDescription(
+        #     key="resumeProgram",
+        #     name="Resume Program",
+        #     icon="mdi:play-pause",
+        # ),
     ),
 }
 
@@ -35,7 +35,7 @@ async def async_setup_entry(hass, entry: ConfigEntry, async_add_entities) -> Non
             hass.data[DOMAIN]["coordinators"][device.mac_address] = coordinator
         await coordinator.async_config_entry_first_refresh()
 
-        if descriptions := BUTTONS.get(device.appliance_type_name):
+        if descriptions := BUTTONS.get(device.appliance_type):
             for description in descriptions:
                 if not device.commands.get(description.key):
                     continue

custom_components/hon/config_flow.py

@@ -11,7 +11,6 @@ _LOGGER = logging.getLogger(__name__)
 
 
 class HonFlowHandler(config_entries.ConfigFlow, domain=DOMAIN):
-
     VERSION = 1
     CONNECTION_CLASS = config_entries.CONN_CLASS_LOCAL_POLL
 

custom_components/hon/hon.py

@@ -28,10 +28,10 @@ class HonEntity(CoordinatorEntity):
     def device_info(self):
         return DeviceInfo(
             identifiers={(DOMAIN, self._device.mac_address)},
-            manufacturer=self._device.brand,
+            manufacturer=self._device.get("brand", ""),
             name=self._device.nick_name if self._device.nick_name else self._device.model_name,
             model=self._device.model_name,
-            sw_version=self._device.fw_version,
+            sw_version=self._device.get("fwVersion", ""),
         )
 
 

custom_components/hon/manifest.json

@@ -0,0 +1,12 @@
+{
+  "domain": "hon",
+  "name": "Haier hOn",
+  "codeowners": ["@Andre0512"],
+  "config_flow": true,
+  "documentation": "https://github.com/Andre0512/hon/",
+  "iot_class": "cloud_polling",
+  "issue_tracker": "https://github.com/Andre0512/hon/issues",
+  "requirements": ["pyhOn==0.3.3"],
+  "version": "0.2.0"
+}
+

custom_components/hon/number.py

@@ -20,19 +20,20 @@ NUMBERS: dict[str, tuple[NumberEntityDescription, ...]] = {
         NumberEntityDescription(
             key="startProgram.delayTime",
             name="Delay Time",
-            icon="mdi:timer",
+            icon="mdi:timer-plus",
             entity_category=EntityCategory.CONFIG,
             native_unit_of_measurement=UnitOfTime.MINUTES
         ),
         NumberEntityDescription(
             key="startProgram.rinseIterations",
             name="Rinse Iterations",
+            icon="mdi:rotate-right",
             entity_category=EntityCategory.CONFIG
         ),
         NumberEntityDescription(
             key="startProgram.mainWashTime",
             name="Main Wash Time",
-            icon="mdi:timer",
+            icon="mdi:clock-start",
             entity_category=EntityCategory.CONFIG,
             native_unit_of_measurement=UnitOfTime.MINUTES
         ),
@@ -52,7 +53,7 @@ async def async_setup_entry(hass, entry: ConfigEntry, async_add_entities) -> Non
             hass.data[DOMAIN]["coordinators"][device.mac_address] = coordinator
         await coordinator.async_config_entry_first_refresh()
 
-        if descriptions := NUMBERS.get(device.appliance_type_name):
+        if descriptions := NUMBERS.get(device.appliance_type):
             for description in descriptions:
                 appliances.extend([
                     HonNumberEntity(hass, coordinator, entry, device, description)]
@@ -66,6 +67,7 @@ class HonNumberEntity(HonEntity, NumberEntity):
         super().__init__(hass, entry, coordinator, device)
 
         self._coordinator = coordinator
+        self._device = device
         self._data = device.settings[description.key]
         self.entity_description = description
         self._attr_unique_id = f"{super().unique_id}{description.key}"
@@ -77,18 +79,18 @@ class HonNumberEntity(HonEntity, NumberEntity):
 
     @property
     def native_value(self) -> float | None:
-        return self._data.value
+        return self._device.get(self.entity_description.key)
 
     async def async_set_native_value(self, value: float) -> None:
-        self._data.value = value
+        self._device.settings[self.entity_description.key].value = value
         await self.coordinator.async_request_refresh()
 
     @callback
     def _handle_coordinator_update(self):
-        self._data = self._device.settings[self.entity_description.key]
-        if isinstance(self._data, HonParameterRange):
-            self._attr_native_max_value = self._data.max
-            self._attr_native_min_value = self._data.min
-            self._attr_native_step = self._data.step
-        self._attr_native_value = self._data.value
+        setting = self._device.settings[self.entity_description.key]
+        if isinstance(setting, HonParameterRange):
+            self._attr_native_max_value = setting.max
+            self._attr_native_min_value = setting.min
+            self._attr_native_step = setting.step
+        self._attr_native_value = setting.value
         self.async_write_ha_state()

custom_components/hon/select.py

@@ -1,4 +1,3 @@
-"""Support for Tuya select."""
 from __future__ import annotations
 
 from pyhon import HonConnection
@@ -11,10 +10,9 @@ from homeassistant.const import UnitOfTemperature, REVOLUTIONS_PER_MINUTE
 from homeassistant.core import callback
 from homeassistant.helpers.entity import EntityCategory
 
+from .const import DOMAIN
 from .hon import HonEntity, HonCoordinator
 
-DOMAIN = "hon"
-
 SELECTS = {
     "WM": (
         SelectEntityDescription(
@@ -52,9 +50,9 @@ async def async_setup_entry(hass, entry: ConfigEntry, async_add_entities) -> Non
             hass.data[DOMAIN]["coordinators"][device.mac_address] = coordinator
         await coordinator.async_config_entry_first_refresh()
 
-        if descriptions := SELECTS.get(device.appliance_type_name):
+        if descriptions := SELECTS.get(device.appliance_type):
             for description in descriptions:
-                if not device.data.get(description.key):
+                if not device.get(description.key):
                     continue
                 appliances.extend([
                     HonSelectEntity(hass, coordinator, entry, device, description)]
@@ -68,32 +66,31 @@ class HonSelectEntity(HonEntity, SelectEntity):
 
         self._coordinator = coordinator
         self._device = device
-        self._data = device.settings[description.key]
         self.entity_description = description
         self._attr_unique_id = f"{super().unique_id}{description.key}"
 
-        if not isinstance(self._data, HonParameterFixed):
-            self._attr_options: list[str] = self._data.values
+        if not isinstance(self._device.settings[description.key], HonParameterFixed):
+            self._attr_options: list[str] = device.settings[description.key].values
         else:
-            self._attr_options = [self._data.value]
+            self._attr_options: list[str] = [device.settings[description.key].value]
 
     @property
     def current_option(self) -> str | None:
-        value = self._data.value
+        value = self._device.settings[self.entity_description.key].value
         if value is None or value not in self._attr_options:
             return None
         return value
 
     async def async_select_option(self, option: str) -> None:
-        self._data.value = option
+        self._device.settings[self.entity_description.key].value = option
         await self.coordinator.async_request_refresh()
 
     @callback
     def _handle_coordinator_update(self):
-        self._data = self._device.settings[self.entity_description.key]
-        if not isinstance(self._data, HonParameterFixed):
-            self._attr_options: list[str] = self._data.values
+        setting = self._device.settings[self.entity_description.key]
+        if not isinstance(self._device.settings[self.entity_description.key], HonParameterFixed):
+            self._attr_options: list[str] = setting.values
         else:
-            self._attr_options = [self._data.value]
-        self._attr_native_value = self._data.value
+            self._attr_options = [setting.value]
+        self._attr_native_value = setting.value
         self.async_write_ha_state()

custom_components/hon/sensor.py

@@ -9,7 +9,7 @@ from homeassistant.components.sensor import (
     SensorEntityDescription,
 )
 from homeassistant.config_entries import ConfigEntry
-from homeassistant.const import UnitOfEnergy, UnitOfVolume, UnitOfMass, UnitOfPower
+from homeassistant.const import UnitOfEnergy, UnitOfVolume, UnitOfMass, UnitOfPower, UnitOfTime
 from homeassistant.core import callback
 from homeassistant.helpers.entity import EntityCategory
 from homeassistant.helpers.typing import StateType
@@ -65,17 +65,30 @@ SENSORS: dict[str, tuple[SensorEntityDescription, ...]] = {
         ),
         SensorEntityDescription(
             key="machMode",
-            name="Machine Last Status",
+            name="Machine Status",
             icon="mdi:information",
             translation_key="mode"
         ),
         SensorEntityDescription(
             key="errors",
-            name="Last Error",
+            name="Error",
             icon="mdi:math-log",
             translation_key="errors"
         ),
-
+        SensorEntityDescription(
+            key="remainingTimeMM",
+            name="Remaining Time",
+            icon="mdi:timer",
+            state_class=SensorStateClass.MEASUREMENT,
+            native_unit_of_measurement=UnitOfTime.MINUTES,
+        ),
+        SensorEntityDescription(
+            key="spinSpeed",
+            name="Spin Speed",
+            icon="mdi:timer",
+            state_class=SensorStateClass.MEASUREMENT,
+            native_unit_of_measurement=UnitOfTime.MINUTES,
+        ),
     )
 }
 
@@ -92,9 +105,9 @@ async def async_setup_entry(hass, entry: ConfigEntry, async_add_entities) -> Non
             hass.data[DOMAIN]["coordinators"][device.mac_address] = coordinator
         await coordinator.async_config_entry_first_refresh()
 
-        if descriptions := SENSORS.get(device.appliance_type_name):
+        if descriptions := SENSORS.get(device.appliance_type):
             for description in descriptions:
-                if not device.data.get(description.key):
+                if not device.get(description.key):
                     continue
                 appliances.extend([
                     HonSensorEntity(hass, coordinator, entry, device, description)]
@@ -114,9 +127,9 @@ class HonSensorEntity(HonEntity, SensorEntity):
 
     @property
     def native_value(self) -> StateType:
-        return self._device.data.get(self.entity_description.key, "")
+        return self._device.get(self.entity_description.key, "")
 
     @callback
     def _handle_coordinator_update(self):
-        self._attr_native_value = self._device.data.get(self.entity_description.key, "")
+        self._attr_native_value = self._device.get(self.entity_description.key, "")
         self.async_write_ha_state()

custom_components/hon/switch.py

@@ -1,13 +1,11 @@
-from collections.abc import Callable, Coroutine
 from dataclasses import dataclass
 from typing import Any
 
-from pyhon import HonConnection
-from pyhon.device import HonDevice
-
 from homeassistant.components.switch import SwitchEntityDescription, SwitchEntity
 from homeassistant.config_entries import ConfigEntry
 from homeassistant.const import EntityCategory
+from pyhon import HonConnection
+from pyhon.device import HonDevice
 
 from .const import DOMAIN
 from .hon import HonCoordinator, HonEntity
@@ -29,20 +27,29 @@ class HonSwitchEntityDescription(HonSwitchEntityDescriptionMixin,
 SWITCHES: dict[str, tuple[HonSwitchEntityDescription, ...]] = {
     "WM": (
         HonSwitchEntityDescription(
-            key="startProgram",
-            name="Start Program",
-            icon="mdi:play",
+            key="active",
+            name="Washing Machine",
+            icon="mdi:washing-machine",
             turn_on_key="startProgram",
             turn_off_key="stopProgram",
         ),
+        HonSwitchEntityDescription(
+            key="pause",
+            name="Pause Washing Machine",
+            icon="mdi:pause",
+            turn_on_key="pauseProgram",
+            turn_off_key="resumeProgram",
+        ),
         HonSwitchEntityDescription(
             key="startProgram.delayStatus",
             name="Delay Status",
+            icon="mdi:timer-check",
             entity_category=EntityCategory.CONFIG
         ),
         HonSwitchEntityDescription(
             key="startProgram.haier_SoakPrewashSelection",
             name="Soak Prewash Selection",
+            icon="mdi:tshirt-crew",
             entity_category=EntityCategory.CONFIG
         ),
     ),
@@ -61,9 +68,9 @@ async def async_setup_entry(hass, entry: ConfigEntry, async_add_entities) -> Non
             hass.data[DOMAIN]["coordinators"][device.mac_address] = coordinator
         await coordinator.async_config_entry_first_refresh()
 
-        if descriptions := SWITCHES.get(device.appliance_type_name):
+        if descriptions := SWITCHES.get(device.appliance_type):
             for description in descriptions:
-                if device.data.get(description.key) is not None or device.commands.get(description.key) is not None:
+                if device.get(description.key) is not None or device.commands.get(description.key) is not None:
                     appliances.extend([
                         HonSwitchEntity(hass, coordinator, entry, device, description)]
                     )
@@ -83,7 +90,7 @@ class HonSwitchEntity(HonEntity, SwitchEntity):
 
     def available(self) -> bool:
         if self.entity_category == EntityCategory.CONFIG:
-            return self._device.settings[self.entity_description.key].typology == "fixed"
+            return self._device.settings[self.entity_description.key].typology != "fixed"
         return True
 
     @property
@@ -92,7 +99,7 @@ class HonSwitchEntity(HonEntity, SwitchEntity):
         if self.entity_category == EntityCategory.CONFIG:
             setting = self._device.settings[self.entity_description.key]
             return setting.value == "1" or hasattr(setting, "min") and setting.value != setting.min
-        return self._device.data.get(self.entity_description.key, "")
+        return self._device.get(self.entity_description.key, False)
 
     async def async_turn_on(self, **kwargs: Any) -> None:
         if self.entity_category == EntityCategory.CONFIG:

custom_components/hon/translations/en.json

@@ -2,7 +2,6 @@
   "config": {
     "step": {
       "user": {
-        "title": "hOn",
         "description": "Please enters your hOn credentials",
         "data": {
           "email": "Email Address",
@@ -15,8 +14,10 @@
     "sensor": {
       "mode": {
         "state": {
+          "0": "Disconnected",
           "1": "Ready",
           "2": "Running",
+          "3": "Paused",
           "5": "Scheduled",
           "6": "Error",
           "7": "Finished"

hacs.json

@@ -1,5 +1,5 @@
 {
-    "name": "hOn",
-    "render_readme": false,
+    "name": "Haier hOn",
+    "render_readme": true,
     "homeassistant": "2023.2.0"
-}
+}