Responsively-app: Improve the App's crash page to have instructions on how to report the issue.

Created on 20 Aug 2020  路  11Comments  路  Source: responsively-org/responsively-app

Right now the crash page just shows the error stacktrace as a string in a single line:
90673436-21b36780-e22e-11ea-964d-2eceb7b88c2c

The improved version can have the following:

  1. Show a clear message that the app has crashed
  2. Add instruction stating a Github issue has to be raised with the stack trace.
  3. Add a copy error button that can copy the error information to the clipboard. And if possible add a link that can take the user to the GitHub create issue page with the content prefilled.
enhancement

All 11 comments

Hello, thanks for this awesome software!
I'm new to contribute and I have no idea about how to show this crash page.
Could you tell me about this?

hi @gorogoroumaru and welcome
from the top of my head: just throw an Error inside any component in the renderer process

Thank you!
I will try.

@gorogoroumaru awesome, should I assign you this issue?

Sorry, @jjavierdguezas.
I couldn't show the crash page.
Could you give me a code example to show this?

@gorogoroumaru Can you please try adding throw new Error('test'); on https://github.com/responsively-org/responsively-app/blob/master/desktop-app/app/components/Addressinput/index.js#L67

Also, please feel free to join your slack workspace for any questions: https://join.slack.com/t/responsively/shared_invite/zt-h0ouywhs-IIeHj39ymz5IOrgcOa_tRA (edited)

@gorogoroumaru the problem is that in development we don't show that page. Go to:

https://github.com/responsively-org/responsively-app/blob/6fd99970e9604fa7a89ca6e5abaf760def48d8a5/desktop-app/app/containers/Root.js#L59

and comment the if statement. Now, you can throw an error (throw new Error('Foo')) from any component.

Let us know if you want to give this issue a try to assign it to you 馃檪

Thank you so much @manojVivek and @jzabala!
I tried your suggested approach, but the screen blacked out and I cannot see the crash page.
Could you help me?
I'm sorry for asking many times.

@gorogoroumaru sure, no worries.

Commenting the if in Root.js

Screen Shot 2020-08-20 at 9 50 34 AM

Throwing error in WebView/index.js

https://github.com/responsively-org/responsively-app/blob/6fd99970e9604fa7a89ca6e5abaf760def48d8a5/desktop-app/app/components/WebView/index.js#L812

I get the error screen:

Screen Shot 2020-08-20 at 9 51 35 AM

Thank you!
I will make a pull request in a few days.
Could you assign me this issue?

great @gorogoroumaru assigned to you

Was this page helpful?
0 / 5 - 0 ratings