Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Collect user data and integrate error page with Intercom #1111

Merged
merged 3 commits into from
Apr 25, 2022
Merged

Collect user data and integrate error page with Intercom #1111

merged 3 commits into from
Apr 25, 2022

Commits on Apr 22, 2022

  1. Collect user data and integrate error page with Intercom 

    This commit puts an ErrorBoundary around each Route's parent component.
This way each ErrorBoundary is under the IntlProvider and has Intercom in scope so we integrate and prefill support request messages when crashes happen.
ErrorBoundary component now gets a `component` prop to specify which component it is wrapping and this will be sent with the errbit report.
It is also collecting user name, dbid and email to send with the report, to help gain more insight/context on the crashes.

Note I: The @airbrake-js/browser calls to the Errbit server seem to ignore the extra data added as context, params and session, unless the error is a string or we cast a new instance from the raised error object. To be investigated. See github issue: airbrake/airbrake-js#1193

Note II: MainErrorBoundary has been renamed to ErrorBoundary

Fixes: CHECK-1744
    @amoedoamorim
    amoedoamorim committed Apr 22, 2022
    Configuration menu
    Copy the full SHA
    e52904a View commit details
    Browse the repository at this point in the history

  2. Remove console log with intentional error 😅 

    Also to mention that change in helpers.js is to guard agains `null` value for `str`.

Fixes CHECK-1747
    @amoedoamorim
    amoedoamorim committed Apr 22, 2022
    Configuration menu
    Copy the full SHA
    40417c8 View commit details
    Browse the repository at this point in the history

  3. Add missing files 🤦

    @amoedoamorim
    amoedoamorim committed Apr 22, 2022
    Configuration menu
    Copy the full SHA
    fdf2f3c View commit details
    Browse the repository at this point in the history