-
Notifications
You must be signed in to change notification settings - Fork 912
Display user friendly messages when errors occur. #8551
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
Conversation
b4bcac1 to
d24015d
Compare
|
/backport to stable-3.17 |
Print the rest of the information about the error in the logs. Signed-off-by: Camila Ayres <hello@camilasan.com>
Signed-off-by: Camila Ayres <hello@camilasan.com>
…not successful. Signed-off-by: Camila Ayres <hello@camilasan.com>
Signed-off-by: Camila Ayres <hello@camilasan.com>
Signed-off-by: Camila Ayres <hello@camilasan.com>
Signed-off-by: Camila Ayres <hello@camilasan.com>
8d2dc87 to
3003a87
Compare
|
Artifact containing the AppImage: nextcloud-appimage-pr-8551.zip SHA256 checksum: To test this change/fix you can download the above artifact file, unzip it, and run it. Please make sure to quit your existing Nextcloud app and backup your data. |
nilsding
left a comment
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.
![]()
|




Before:






After:
ℹ️ I think the huge red box with the error for each file is not ideal, needs to change, just not in this PR: