Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Mar 14, 2022

Conversation

skrashevich
Copy link
Contributor

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

@alryaz
Copy link
Owner

alryaz commented Mar 14, 2022

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

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

@alryaz
Copy link
Owner

alryaz commented Mar 14, 2022

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

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

@skrashevich
Copy link
Contributor Author

skrashevich commented Mar 14, 2022

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

@alryaz
Copy link
Owner

alryaz commented Mar 14, 2022

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

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

@alryaz alryaz merged commit 52b6ee8 into alryaz:main Mar 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants