vooon / hass-myheat

Home Assistant intergration with myheat.net gas heating controllers
MIT License
9 stars 3 forks source link

Missed icon #24

Closed pdacity closed 8 months ago

pdacity commented 1 year ago

Не отображается иконка интеграции.

GET https://brands.home-assistant.io/myheat/icon.png 404

изображение

И наверное логичней будет использовать такую: mdi:water-boiler

изображение

vooon commented 1 year ago

Я ее и не загружал. Иконки интеграций лежат совершенно отдельно, обычно там товарный знак производителя, но сомневаюсь, что MyHeat мне разрешат залить их иконку.

Ну и для начальных версий плагина я даже не думал об этом. Куда важнее поддержать фичи контроллера.

vooon commented 1 year ago

Screenshot 2023-02-04 at 10-34-33 Настройки – Home Assistant (pixelized)

Вот тут к примеру интеграция с колонкой уже зрелая, а по началу на сколько помню тоже без иконки была. Возможно важнее опубликовать компонент в реестре HACS, чтоб не приходилось добавлять репу вручную.

pdacity commented 1 year ago

В HACS прям годная мысль. А иконка - ну можно взять готовую из библиотеки, примерно корелируется с задачей. В ишью есть название.

pdacity commented 9 months ago

Про иконку интеграции. https://community.home-assistant.io/t/custom-component-integration-icon/191726/2

vooon commented 9 months ago

Так по прежнему нужно загружать иконку в репу брэндов. property icon - это про другое - про иконку уже entity, например switch.

Я вообще задал производителю вопрос по поводу добавления их иконки, но пока молчат.

vooon commented 9 months ago

Производитель дал разрешение. Теперь осталось дождаться: https://github.com/home-assistant/brands/pull/5216

vooon commented 8 months ago

Готово!

vooon commented 8 months ago

Еще запросил добавление в список интеграций хасса: https://github.com/hacs/default/pull/2379