Skip to content

Add support for volume level - #28

Open
DestruktorGitHub wants to merge 4 commits into
parautenbach:masterfrom
DestruktorGitHub:VolumeLevel
Open

DestruktorGitHub wants to merge 4 commits into
parautenbach:masterfrom
DestruktorGitHub:VolumeLevel

Conversation

@DestruktorGitHub

Copy link
Copy Markdown

I also tried to implement the ability to set the volume level, but it turned out that this is no longer supported in newer versions of iOS, so I left only the functions responsible for displaying the current Airplay volume, which may make the code look a little messy.

A minor fix that prevents an error from being displayed when the user tries to use the slider.
@charlesomer

Copy link
Copy Markdown

This looks like it would close: #27 too. Just tested it on my setup and at first glance it appears to work. Nice! Although I imagine it's going to confuse people when setting the volume via HA doesn't work so some documentation might be needed 😄

@parautenbach

Copy link
Copy Markdown
Owner

Thank you for the PR!

Although I imagine it's going to confuse people when setting the volume via HA doesn't work so some documentation might be needed

Agreed.

Please also bump the version.

@DestruktorGitHub

Copy link
Copy Markdown
Author

Okay, I added a Polish translation for the config flow, changed the README file to explain the current situation (why remote control from HA is currently not working), and listed the new features in the CHANGELOG file.

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.

3 participants