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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump vlc_telnet to 2.0.1, several bug fixes #44776
Conversation
Switch to python-telnet-vlc 2.0.1 and avoid some unnecessary int/str conversions. Add SUPPORT_VOLUME_STEP. Fix various state issues (initial state not sent as part of first update, unmute detection, seeking didn't actually work).
Please split this PR into at least three PRs:
|
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.
Please split this PR into at least three PRs:
- Bump library version
- Fix issues
- Add volume step support
Marking this as "Request changes"
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. |
Sorry for the delay, finally had cycles to test what happens if we slice it up as requested. Unfortunately, while Which course of action would you like to pursue?
|
Please do 3. |
Alright, I've extracted the first of several PRs into #46608 (apologies if there's a better way to do this in git). (I'm planning to cancel this PR once the other PRs are in) |
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. |
Seems like 3 PR are made and merged. As far as I can see, the update handling changes have not ended up in the other PRs. Is there still a PR on route? Or are you planning on rebasing/updating this PR? |
Hey @frenck, thanks for checking in. Right, update handling changes still haven't been pulled out and I'm still planning on getting to this (there might be others too). That said, there's probably no point in keeping this one open since it's not going anywhere, so will close it. |
Switch to python-telnet-vlc 2.0.1 and avoid some unnecessary int/str
conversions.
Add
SUPPORT_VOLUME_STEP
.Fix various state issues (initial state not sent as part of first
update, unmute detection, seeking didn't actually work).
Breaking change
Proposed change
Type of change
Example entry for
configuration.yaml
:# Example configuration.yaml
Additional information
Checklist
black --fast homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.The integration reached or maintains the following Integration Quality Scale:
To help with the load of incoming pull requests: