feat: adds support for reporting unknown devices.

- adds the ThreeSixty2 to the correct type
- looks for supported types from know values
- adds issue template and HA issue reporting

Author: SSmale

.github/ISSUE_TEMPLATE/device_not_supported.md

@@ -0,0 +1,50 @@
+---
+name: Device Not Supported
+about: This is to submit details of a device not being supported.
+title: ''
+labels: User Reported
+assignees: ''
+body:
+  - id: device_name
+    type: textarea
+    attributes:
+      label: The device name from Duux
+  - id: device_type_id
+    type: textarea
+    attributes:
+      label: The device type ID
+  - id: sensor_type_id
+    type: textarea
+    attributes:
+      label: The sensor type ID
+  - id: reported_type
+    type: textarea
+    attributes:
+      label: The device type Duux reports to Google
+---
+
+<!-- Please aim to complete all sections -->
+
+**Describe the device**
+
+<!-- A clear and concise description of what the device is. A URL to the product page and manual would be helpful -->
+
+**Screenshots**
+
+<!-- Screenshots of the device as shown in the Duux app -->
+
+**Supported Commands**
+
+<!-- Please paste the output from following this guide: https://github.com/SSmale/Duux-Home-Assistant/wiki/Getting-the-sensor-data-from-Home-Assistant -->
+
+```
+{}
+```
+
+**Additional context**
+
+<!-- Add any other context about the problem here. -->
+
+**Are you willing to help test?**
+
+<!-- Getting the device setup properly requires some testing, are you up for helping? -->

custom_components/duux/__init__.py

@@ -7,13 +7,16 @@
 from homeassistant.const import Platform
 from homeassistant.core import HomeAssistant
 from homeassistant.helpers.update_coordinator import DataUpdateCoordinator, UpdateFailed
+from homeassistant.helpers import issue_registry as ir
+
 
 from .const import (
     DOMAIN,
     DUUX_DTID_HEATER,
     DUUX_DTID_THERMOSTAT,
     DUUX_DTID_HUMIDIFIER,
     DUUX_DTID_OTHER_HEATER,
+    DUUX_SUPPORTED_TYPES,
 )
 from .duux_api import DuuxAPI
 
@@ -48,22 +51,55 @@ async def async_setup_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
     for device in devices:
         sensor_type_id = device.get("sensorTypeId")
         device_type_id = device.get("sensorType").get("type")
+        google_type = device.get("sensorType").get("googleDeviceType")
+        last_word = google_type.split(".")[-1]  # "HEATER" OR ""THERMOSTAT"
+        device_name = device.get("displayName") or device.get("name")
+        model = device.get("sensorType", {}).get("name", "Unknown")
+
         if device_type_id not in [
             *DUUX_DTID_HEATER,
             *DUUX_DTID_THERMOSTAT,
             *DUUX_DTID_HUMIDIFIER,
             *DUUX_DTID_OTHER_HEATER,
         ]:
+            ir.async_create_issue(
+                hass,
+                DOMAIN,
+                "device_not_recognised",
+                is_fixable=False,
+                severity=ir.IssueSeverity.WARNING,
+                translation_key="device_not_recognised",
+                learn_more_url=f"https://github.com/SSmale/Duux-Home-Assistant/issues/new?template=device_not_recognised.md&title=Device+not+recognised+-+{model}&device_name={model}&device_type_id={device_type_id}&sensor_type_id={sensor_type_id}&reported_type={google_type}",
+                translation_placeholders={
+                    "device_name": model,
+                    "device_type_id": device_type_id,
+                    "sensor_type_id": sensor_type_id,
+                    "reported_type": google_type,
+                },
+            )
+            _LOGGER.warning("Your device has not been recognised.")
+            _LOGGER.warning(
+                f"It is classified as type {last_word}, so maybe loaded as such.",
+            )
+            _LOGGER.warning(
+                "Please report this to the integration developer so they can update the supported device list.",
+            )
             _LOGGER.warning(
-                f"Unknown device type {device_type_id}:{sensor_type_id}, skipping.."
+                f"Required details: Device Name: {model}, Device Type ID: {device_type_id}, Sensor Type ID: {sensor_type_id}, Google Device Type: {google_type}",
             )
-            continue
+            if last_word in DUUX_SUPPORTED_TYPES:
+                _LOGGER.warning(
+                    f"Attempting to load device as type {last_word}.",
+                )
+
+            else:
+                continue
 
         coordinator = DuuxDataUpdateCoordinator(
             hass,
             api=api,
             device_id=device.get("deviceId"),
-            device_name=device.get("displayName") or device.get("name"),
+            device_name=device_name,
         )
 
         await coordinator.async_config_entry_first_refresh()

custom_components/duux/climate.py

@@ -18,6 +18,7 @@
 from homeassistant.helpers.update_coordinator import CoordinatorEntity
 
 from .const import (
+    DUUX_CLIMATE_TYPES,
     DUUX_DTID_THERMOSTAT,
     DUUX_DTID_HEATER,
     DOMAIN,
@@ -44,12 +45,32 @@ async def async_setup_entry(
     entities = []
     for device in devices:
         device_type_id = device.get("sensorType").get("type")
-        if device_type_id not in [*DUUX_DTID_HEATER, *DUUX_DTID_THERMOSTAT]:
-            continue
-
+        google_type = device.get("sensorType").get("googleDeviceType")
+        last_word = google_type.split(".")[-1]  # "HEATER" OR ""THERMOSTAT"
         sensor_type_id = device.get("sensorTypeId")
         device_id = device["deviceId"]
         coordinator = coordinators[device_id]
+
+        model = device.get("sensorType", {}).get("name", "Unknown")
+
+        if device_type_id not in [*DUUX_DTID_HEATER, *DUUX_DTID_THERMOSTAT]:
+            if last_word in DUUX_CLIMATE_TYPES:
+                _LOGGER.warning(
+                    "Your device has not been officially catagorised as supporting the climate platform."
+                )
+                _LOGGER.warning(
+                    f"It is classified as type {last_word}, so attempting to set up as a climate device.",
+                )
+                _LOGGER.warning(
+                    "Please report this to the integration developer so they can update the supported device list.",
+                )
+                _LOGGER.warning(
+                    f"Required details: Device Name: {model}, Device Type ID: {device_type_id}, Sensor Type ID: {sensor_type_id}, Google Device Type: {google_type}",
+                )
+
+            else:
+                continue
+
         # Create the appropriate climate entity based on heater type
         if sensor_type_id == DUUX_STID_THREESIXTY_2023:
             entities.append(DuuxThreesixtyClimate(coordinator, api, device))
@@ -68,6 +89,7 @@ async def async_setup_entry(
 
     async_add_entities(entities)
 
+
 class DuuxClimate(CoordinatorEntity, ClimateEntity):
     """Representation of a Duux climate device."""
 
@@ -95,7 +117,7 @@ def __init__(self, coordinator, api, device):
             | ClimateEntityFeature.TURN_OFF
             | ClimateEntityFeature.TURN_ON
         )
-    
+
     @property
     def device_info(self):
         """Return device information."""
@@ -105,7 +127,7 @@ def device_info(self):
             "manufacturer": self._device.get("manufacturer", "Duux"),
             "model": self._device.get("sensorType", {}).get("name", "Unknown"),
         }
-    
+
     @property
     def current_temperature(self):
         """Return the current temperature."""
@@ -121,7 +143,7 @@ def hvac_mode(self):
         """Return current operation."""
         power = self.coordinator.data.get("power", 0)
         return HVACMode.HEAT if power == 1 else HVACMode.OFF
-    
+
     @property
     def preset_mode(self):
         """Return current preset mode."""
@@ -133,7 +155,7 @@ def preset_modes(self):
         """Return available preset modes."""
         # Base implementation - override in subclasses
         return []
-    
+
     async def async_set_temperature(self, **kwargs):
         """Set new target temperature."""
         if (temperature := kwargs.get(ATTR_TEMPERATURE)) is None:

custom_components/duux/const.py

@@ -19,13 +19,16 @@
 DUUX_STID_BORA_2024 = 62
 
 # Device Type IDs
-
 DUUX_DTID_THERMOSTAT = [50, 52]
-DUUX_DTID_HEATER = [51]
+DUUX_DTID_HEATER = [51, 21]
 DUUX_DTID_HUMIDIFIER = [56]
 
-DUUX_DTID_OTHER_HEATER = [21, 23]
+DUUX_DTID_OTHER_HEATER = [23]
+
+DUUX_CLIMATE_TYPES = ["THERMOSTAT", "HEATER"]
+DUUX_HUMIDIFIER_TYPES = ["HUMIDIFIER"]
 
+DUUX_SUPPORTED_TYPES = DUUX_CLIMATE_TYPES + DUUX_HUMIDIFIER_TYPES
 
 # {
 #     "data": [

custom_components/duux/translations/en.json

@@ -0,0 +1,8 @@
+{
+    "issues": {
+        "device_not_recognised": {
+            "description": "Duux has detected a device that is not officially supported. \n\rPlease check the device type and report it to the integration developer. \n\rDetails needed are: \n\rDevice Name: {device_name}\n\rDevice Type ID: {device_type_id}\n\rSensor Type ID: {sensor_type_id}\n\rDevice Type: {reported_type}.",
+            "title": "Device not recognised"
+        }
+    }
+}