Give reason for failure to load instead of "please try again later"
Is your feature request related to a problem? Please describe.
Several bug reports such as this one #2778 are related to a failure to load Nextcloud office/collabora. The user is advised to "try again later", which makes little sense if there is a configuration error that won't go away by trying again later. In the case of #2278, Collabora refuses the connection because it does not know the WOPI host. In such a case , there should be a meaningful error message appearing in the browser, instead of the Microsoftish "try again later. This would, I think, prevent the users from just fumbling around in the dark.
Describe the solution you'd like
If the Collabora host refuses the connection, this should be said in the error message on screen.
Describe alternatives you've considered
Yes, sure, if you examine the nextcloud.log, you find a hint there, but the nextcloud log is rather overwhelming, you easily miss the forest for the trees. Therefore I'd prefer the simple and concise message I have suggested.
Additional context
Would also reduce the load on the bug tracking system, I guess.
Yes, that would be very helpful.
Just earlier this week I've discussed with @kendy about having a very early post message and also post messages about potential error cases.
@kendy Is there any list of failure scenarios that would emit an error (with an existing post message or maybe new ones) so we could act accordingly and give the users better feedback on what went wrong?
@juliushaertl: Let me add @rash419 who has implemented this feature - Rash, can you help, please?
we don't have it now, but should be possible & we have added it to our planning Thanks
Looks like these are the ones currently available:
https://github.com/CollaboraOnline/online/blob/c0cdbd8d740d05cfd1c5d2a9a67bf61ad4b163a9/wsd/Storage.cpp#L240-L277
And could be caught and passed along here I think:
https://github.com/CollaboraOnline/online/blob/c0cdbd8d740d05cfd1c5d2a9a67bf61ad4b163a9/wsd/DocumentBroker.cpp#L1002-L1012