thinknimble / tn-spa-bootstrapper

A production-ready Django SPA app on Heroku in 20 minutes or less!
https://tn-spa-bootstrapper-staging.herokuapp.com/
MIT License
8 stars 9 forks source link

Sentry Crash #332

Open paribaker opened 3 months ago

paribaker commented 3 months ago

It seems we do not always get the sentry crash reports for example every once in a while we have an issue where the app crashes on load, I believe this may be because we need to move it higher in the tree? Right now Sentry depends on the Config entry file we have which means any issues that occur in that file may not be reported.

whusterj commented 1 week ago

I just experienced this on a project. I think it's because SENTRY_PROJECT_NAME and SENTRY_DSN are not part of the expo-main.yml GH action template. So when that action runs eas update, it pushes a new OTA update with a JS bundle that is missing those values. And so Sentry crashes the app on startup.

I verified this by sourcing all env variables on my local and then running the same eas update command found in expo-main.yml.

Aside, I would love to remove the mobile bootstrapper's hard dependency on Sentry and Rollbar. I think there's too much friction in bootstrapping a new mobile app.