search

alryaz / hass-yandex-music-browser

Yandex Media Browser
64 stars 9 forks source link

Исправление ошибок запуска. Удалён вход по паролю #8

Closed skrashevich closed 2 years ago

skrashevich commented 2 years ago

Теперь работает с HA >= 2022.3 Входа через пароль больше нет. Зависимость от интеграции YandexStation стала практически безусловной для рядового пользователя, соответствующие изменения внесены в readme.

alryaz commented 2 years ago

Боюсь, что искоренение конкретизации версии основной библиотеки (PIP: yandex-music) из раздела зависимостей в манифесте может иметь отрицательные последствия (до версии 1.0.0 пакет не раз менял некоторые основные методы и поля).

Сейчас перепроверю изменения.

alryaz commented 2 years ago

@skrashevich проверьте, пожалуйста, при версии 2.0.1 (yandex-music~=2.0.1), и если это сработает, пожалуйста, внесите изменения в манифест.

Для искоренения авторизации через логин/пароль ещё надо немного кода убрать, но это в грядущем времени (требуется всё же переписать компонент полностью в соответствие с архитектурой HA).

skrashevich commented 2 years ago

Начиная как раз с 2022.3 в HA жесткая проверка зависимостей, поэтому прибивать конкретную версию - не лучшая идея. Текущий код работает как на yandex-music 1.0.0, так и на 2.* Логичнее тогда указать >=1.0.0 в манифесте

alryaz commented 2 years ago

Тут как раз речь о ситуации ≥2.01, ==2. (<3.).

Тут вряд ли жёсткая проверка зависимостей, а, скорее, появление коллизий (судя из описаний ошибки #7). Отступление от == действительно позволит обновляться до любого совместимого релиза в пределах 2.*.