-
Notifications
You must be signed in to change notification settings - Fork 804
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
Clear error list after displaying it once. #6831
base: master
Are you sure you want to change the base?
Conversation
66e8a73
to
0c65513
Compare
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.
By doing this are we not wiping away ay errors that need actions from users?
I also don't see how this fixes the errors referenced in the PR post; it seems to me we should be more lenient in how we tolerate post-device-wake-up errors in the client, rather than changing how errors are presented
0c65513
to
8011c30
Compare
Yes, I am still thinking on how to properly/correctly approach this. |
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
Fix for #6448 and #6487. Signed-off-by: Camila Ayres <[email protected]>
Signed-off-by: Camila Ayres <[email protected]>
Signed-off-by: Camila Ayres <[email protected]>
8011c30
to
e575561
Compare
AppImage file: nextcloud-PR-6831-e57556138b75a1e579ea0a79db6cd89ef9286632-x86_64.AppImage |
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
Possible fix for #6448 and #6487.