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

Add new configuration option: "Require confirmation on quit" #230

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

bhughes339
Copy link

My use case for SonoBus is an always-on bridge between my home and work PCs for my audio interface. It allows me to use the same headphones and microphone for both systems seamlessly.

Because SonoBus is always on and connected to a private group, if I try to restart one of the PCs and forget it's connected (which happens more than I'd like to admit), the restart is cancelled due to the confirmation popup. This change adds a small option to disable that popup. It defaults to true so there's no breaking usability changes.

@essej
Copy link
Collaborator

essej commented Jun 25, 2024

I like this, sorry for the late review!

It should only be shown in the Standalone version however, so some of those items should move into the standalone only conditionals...

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