-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Properly show locked file error message in web UI #16701
Comments
This will require fixing all the ajax calls of the files web UI to properly catch exceptions and actually return them: (something which seems to have been missing for a long while) |
PR here #16718 |
The PR above fixed many operations but not "Download". |
Download case here #17009 |
do we really need this? what's the impact? THX |
Probably not needed. |
Whenever a file is locked, instead of hanging, the web UI needs to show a proper error message.
Currently the network console will return "500 Internal server error" and the log will show that there was a LockingException.
The LockingException needs to be caught properly in the ajax endpoints to be translated into a proper error message.
Needed to be able to show the locking exceptions:
The text was updated successfully, but these errors were encountered: