-
Notifications
You must be signed in to change notification settings - Fork 435
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
Cannot disable sip #4951
Cannot disable sip #4951
Conversation
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
/backport to stable20.1 |
because they grew from some initial confusion and eventually it seems I only adjusted a single one later on |
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.
👍
Which is good and which is bad? so I can use either and do a string replace? |
The backport to stable20.1 failed. Please do this backport manually. |
I don't remember. But one of them might not properly reset the checkbox state in case of error. |
I had some WIP for error handling and trying to reset the states here: #4671 |
Seems this is only broken in 21, works fine in 20. |
🙈 I wonder why
LockingSettings.vue
andSipSettings.vue
are so different when they do the same thing :D