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

Webapp2 lock volume #510

Closed
wants to merge 3 commits into from
Closed

Webapp2 lock volume #510

wants to merge 3 commits into from

Conversation

SteveMicroNova
Copy link
Contributor

Adding a lock to the volume sliders on the frontend (and logic to support it to the backend) that will allow volume to be locked so that it can't be changed, generally useful when a zone is part of a group but doesn't want to be effected by the same volume changes as the group as a whole

@SteveMicroNova SteveMicroNova linked an issue Sep 1, 2023 that may be closed by this pull request
@SteveMicroNova
Copy link
Contributor Author

Abandoning this branch temporarily to work on higher priority things, but this will be a good feature to have once I get back to it

Base automatically changed from webapp_2 to develop October 3, 2023 20:50
@SteveMicroNova
Copy link
Contributor Author

This branch is likely stale and will need to be rebased/restarted whenever I get back to it
This proved to be a larger undertaking than assumed, needing to bake the volume lock functionality directly into the core of amplipi as well as every individual stream type rather than just having a frontend toggle for it

@rtertiaer rtertiaer deleted the branch main December 21, 2023 21:33
@rtertiaer rtertiaer closed this Dec 21, 2023
@rtertiaer
Copy link
Contributor

@SteveMicroNova if you intend to work on this, can you re-target this PR to main? Github automagically closed it when I deleted develop 🥴 sorry!

@linknum23 linknum23 reopened this Dec 22, 2023
@linknum23 linknum23 changed the base branch from develop to main December 22, 2023 15:12
@linknum23
Copy link
Contributor

My instincts are to close this until we decide how to implement this. Lets wait for more feedback on the new UI.

@linknum23 linknum23 closed this Dec 29, 2023
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.

Add a "Volume Lock" setting to zones/groups in the player page
3 participants