-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Merge upstream from dilruacs repo #37
base: master
Are you sure you want to change the base?
Conversation
…use of external file
Added remote entity
# Conflicts: # README.md # custom_components/sony/manifest.json # custom_components/sony/media_player.py # test_config/create_docker.sh
# Conflicts: # custom_components/sony/media_player.py
There are a lot of cool changes in here. If you can get this into a working state, it might be worth the effort to get this integrated into HACS |
Improve volume supprt * Add volume level * Change volume debug level * Add volume set support & normalize volume (cherry picked from commit 4142b1f)
"documentation": "https://github.com/alexmohr/media_player.sony", | ||
"dependencies": [], | ||
"codeowners": [], | ||
"version": "0.1.1", | ||
"requirements": ["sonyapilib==0.5.0"] | ||
"requirements": ["sonyapilib==0.5.0"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With the current master, volume functions are broken because we require 0.5.0 which does not include the new functions yet.
Before accepting this, a new version in sonyapilib has to be released. This comment is as reminder to do that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't be better to release the new version of sonyapilib now to fix currently broken master?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be better but I don't have time to do so at the moment, so it stays broken until then.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No problem, just saying
Seems to be working, but needs more testing and I have found some bugs which will try to fix later.
Just concept for now :D