Ios: Collabora view broken if Collabora not available

Created on 17 Dec 2018  Â·  14Comments  Â·  Source: nextcloud/ios

  • having an abc.odt document in your Nextcloud
  • having Collabora app enabled in your server
  • having the collabora service not running (for whatever reason) on the server, so that the reverse proxy returns a 502 return code
  • click on the abc.odt in the iOS app
  • expected: I get a nice warning with "Can't reach the Collabora service - try again later" and a back button
  • actual: completely white screen without any button or way to go back and []

This is how it looks. The only way out is to kill the app forcefully:

img_0431

cc @marinofaggiana

bug

All 14 comments

@MorrisJobke, I tested the return of createLinkRichdocuments ... your test is strange ... the end point createLinkRichdocuments return yes but do not open file ? can you give me an account for test this condition?

@MorrisJobke, I tested the return of createLinkRichdocuments ... your test is strange ... the end point createLinkRichdocuments return yes but do not open file ? can you give me an account for test this condition?

I send them via e-mail to you ;)

@tobiasKaminsky have you resolved this issue ?

This is only in the iOS app or did I miss something?

I don't know .. but I want know from @tobiasKaminsky how (and if) he detect the collabora page

I fear that this will also happen on android.
We simply call richDocuments to get us a link. And then we are opening this link.
On Android it might look a bit nicer as we show a spinner until COOL is loaded (which then will never be).
And clicking back will also closes the webview.

But I do not see a quick possibility how to detect if the given link from rich documents app is valid and working, other than trying it…

This is fine, but it should be a better looking error than a white page without any button

@rullzer the direct link you generate returns [] and 204 (empty content).
This is not really possible to detect with android.
However any status >400 (error) would be a lot easier to detect.
Can this be changed?

For inspiration, @marinofaggiana
https://nextcloud.kaminsky.me/index.php/s/oFBDWoHtaZqsgqJ

Thanks Toby, already exist the activity when loading the Collabora but this issue is for:

@rullzer the direct link you generate returns [] and 204 (empty content).
This is not really possible to detect with android.

and with iOS .. @rullzer and this : [] ... ??

However any status >400 (error) would be a lot easier to detect.
Can this be changed?

yes ! Correct.

@marinofaggiana I am showing the loading spinner / screen as long until I receive via JS bridge "documentLoaded". As this will never be sent on a broken Collabora, it is at least a bit more "error handling".

With https://github.com/nextcloud/richdocuments/pull/382 I now get the "502 Bad Gateway" page of Nginx. cc @juliushaertl

Ah so we need to check if the actual collabora frame is loading properly as well. :thinking:

@marinofaggiana Any update here - still broken.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

AxelGard picture AxelGard  Â·  4Comments

khlschrnk picture khlschrnk  Â·  5Comments

marinofaggiana picture marinofaggiana  Â·  5Comments

MorrisJobke picture MorrisJobke  Â·  4Comments

sakthi-d picture sakthi-d  Â·  4Comments