search

home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
71.11k stars 29.79k forks source link

OpenTherm device_info property missing #28378

Closed andriej closed 4 years ago

andriej commented 4 years ago

Home Assistant release with the issue: 0.101.0

Last working Home Assistant release (if known): n/a (integration since 0.101)

Operating environment (Hass.io/Docker/Windows/etc.): Linux venv

Integration: https://www.home-assistant.io/integrations/opentherm_gw/

Description of problem: since this version OTGW moved to integrations with config flow. All entities made by the integration (except climate domain) are missing device_info property, thus are missing from entities register and cannot be turned off in example.

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant): n/a

Traceback (if applicable): no traceback

Additional information: n/a

probot-home-assistant[bot] commented 4 years ago

Hey there @mvn23, mind taking a look at this issue as its been labeled with a integration (opentherm_gw) you are listed as a codeowner for? Thanks!

mvn23 commented 4 years ago

Thanks for the report, I will have time to take a look later this weekend.

mvn23 commented 4 years ago

Basic device info support is completed, after I sort out support for re-enabling disabled entities I will file a PR.

jodur commented 4 years ago

The config-flow introduced in 101.3 for this component is also buggy. The climate entity is not available anymore after updating to this version. Also removing it from config-file or removing en re-adding doesn't work. I already saw 2 person with the same problem.

mvn23 commented 4 years ago

@jodur if you mean #28363 then a fix is in the pipeline. If not, then please file a separate report with as much information as possible.