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

"Set all faders to new client level" only affects current users #1420

Open
pgScorpio opened this issue Apr 1, 2021 · 2 comments · May be fixed by #3151
Open

"Set all faders to new client level" only affects current users #1420

pgScorpio opened this issue Apr 1, 2021 · 2 comments · May be fixed by #3151
Labels
feature request Feature request
Milestone

Comments

@pgScorpio
Copy link
Contributor

Describe the bug

When using "Set all faders to new client level" this only resets the faders of current users, if a previous user signs in it's level is still set to the previous level.

To Reproduce

Sign in to a server with multiple users.
Set all levels to i.e. max.
Sign out one user.
use "Set all faders to new client level".
sign in the previous user again and it's level will still be set to max. not the "New Client Level".

Expected behavior

Just like when using "Clear All Stored Solo and Mute Settings" also the stored levels should be reset to new client level when using "Set all faders to new client level"

Screenshots

Operating system

n.a.

Version of Jamulus

3.6.x, 3.7.0
Additional context

Maybe also an added option to delete all stored users?

@pgScorpio pgScorpio added the bug Something isn't working label Apr 1, 2021
@gilgongo gilgongo added this to Triage in Tracking (old) via automation Apr 2, 2021
@gilgongo
Copy link
Member

gilgongo commented Apr 2, 2021

I believe this is by design (there is a discussion about this point somewhere...), but I agree it feels like a bug.

@pgScorpio
Copy link
Contributor Author

I think "Set all faders to new client level" should also remove all "storedfader" values in the ini file of not currently connected clients, In that way we can keep the inifile tidy and it will also apply the current new client level when an old client joins again.

@pljones pljones added this to the Release 3.11.0 milestone Oct 5, 2022
@pljones pljones moved this from Triage to Backlog in Tracking (old) Oct 5, 2022
@pljones pljones removed this from Backlog in Tracking (old) Jun 17, 2023
pljones added a commit to pljones/jamulus that referenced this issue Aug 1, 2023
pljones added a commit to pljones/jamulus that referenced this issue Aug 1, 2023
pljones added a commit to pljones/jamulus that referenced this issue Aug 3, 2023
@pljones pljones self-assigned this Aug 12, 2023
pljones added a commit to pljones/jamulus that referenced this issue Aug 15, 2023
pljones added a commit to pljones/jamulus that referenced this issue Aug 23, 2023
@pljones pljones linked a pull request Aug 23, 2023 that will close this issue
8 tasks
pljones added a commit to pljones/jamulus that referenced this issue Aug 25, 2023
pljones added a commit to pljones/jamulus that referenced this issue Aug 28, 2023
pljones added a commit to pljones/jamulus that referenced this issue Aug 28, 2023
pljones added a commit to pljones/jamulus that referenced this issue Sep 5, 2023
pljones added a commit to pljones/jamulus that referenced this issue Sep 12, 2023
@pljones pljones removed their assignment Sep 18, 2023
pljones added a commit to pljones/jamulus that referenced this issue Sep 18, 2023
@pljones pljones added feature request Feature request and removed bug Something isn't working labels Sep 18, 2023
pljones added a commit to pljones/jamulus that referenced this issue Sep 27, 2023
pljones added a commit to pljones/jamulus that referenced this issue Oct 2, 2023
pljones added a commit to pljones/jamulus that referenced this issue Oct 5, 2023
pljones added a commit to pljones/jamulus that referenced this issue Oct 15, 2023
pljones added a commit to pljones/jamulus that referenced this issue Nov 2, 2023
pljones added a commit to pljones/jamulus that referenced this issue Dec 9, 2023
pljones added a commit to pljones/jamulus that referenced this issue Dec 14, 2023
pljones added a commit to pljones/jamulus that referenced this issue Jan 14, 2024
pljones added a commit to pljones/jamulus that referenced this issue Feb 12, 2024
pljones added a commit to pljones/jamulus that referenced this issue Feb 18, 2024
pljones added a commit to pljones/jamulus that referenced this issue Mar 5, 2024
pljones added a commit to pljones/jamulus that referenced this issue Mar 15, 2024
pljones added a commit to pljones/jamulus that referenced this issue Mar 28, 2024
pljones added a commit to pljones/jamulus that referenced this issue May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Feature request
Projects
Status: Triage
Development

Successfully merging a pull request may close this issue.

3 participants