Skip to content

[CHORE] Bugsnag logging setup#1145

Merged
diegolmello merged 40 commits intodevelopfrom
chore-add-bugsnag
Aug 23, 2019
Merged

[CHORE] Bugsnag logging setup#1145
diegolmello merged 40 commits intodevelopfrom
chore-add-bugsnag

Conversation

@IlarionHalushka
Copy link
Copy Markdown
Contributor

@IlarionHalushka IlarionHalushka commented Aug 22, 2019

@RocketChat/ReactNative
Closes #123

Previous PR for configuring bugsnag was closed because it was created in IlarionHalushka's repository and this new PR is created in Rocket.Chat.ReactNative repo.

This PR aims to add error logging using Bugsnag.

Bugsnag provides ability to upload source maps for each build in order to be able to see an informative stack trace.

  • basic configuration of bugsnag
  • upload iOS source maps in ios-build CI job
  • upload Android source maps in android-build CI job
  • exchange firebase error logging to bugsnag
  • add to Settings Send crash reports switch which enables\disables reports\analytics sending.
  • change API key from Ilarion's to RocketChat's and test error reporting on release builds

Screenshots of error stack trace on Bugsnag:
Screen Shot 2019-08-01 at 10 59 35 PM

Uploaded source maps (release via CI, debug from local machine):
Screen Shot 2019-08-02 at 11 01 44 AM

IlarionHalushka and others added 27 commits August 1, 2019 10:21
# Conflicts:
#	.circleci/config.yml
#	android/build.gradle
#	app/views/SettingsView/index.js
#	ios/Podfile
#	ios/RocketChatRN.xcodeproj/project.pbxproj
#	package.json
#	yarn.lock
Copy link
Copy Markdown
Member

@diegolmello diegolmello left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm in love with this PR.
Thanks, @IlarionHalushka!

@diegolmello diegolmello merged commit 59a71ff into develop Aug 23, 2019
@diegolmello diegolmello deleted the chore-add-bugsnag branch August 23, 2019 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add some error tracking like crashlytics or bugsnag

2 participants