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.

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

Setup Firebase Crashlytics #568

Merged
merged 9 commits into from
May 19, 2024
Merged

Setup Firebase Crashlytics #568

merged 9 commits into from
May 19, 2024

Conversation

prabhanshuguptagit
Copy link
Contributor

@prabhanshuguptagit prabhanshuguptagit commented May 17, 2024

This sets up Firebase and reports crashes to Crashlytics. Link to dashboard.

Notes

  • I was only able to build the prod flavor (--flavor prod), got this error otherwise
Execution failed for task ':app:processStagingDebugGoogleServices'.
> No matching client found for package name 'org.tramline.ueno.staging'
  • It seems like it's ok to commit google-service.json and firebase_options.dart to public repositories (from here and here) so I've left them in for convenience. I can re-configure and wipe before merge if we don't want them public.

  • The events are being logged under the Ueno style app. I've tested it by throwing a few exceptions. I'm not sure why it picked that one over Ueno android – I suspect it has something to do with the application ID in build.gradle.

Copy link
Member

@kitallis kitallis left a comment

Choose a reason for hiding this comment

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

Thx ❇️

Tested it locally, works.

I also added a random non-fatal error at boot that gets sent to both Bugsnag + Crashlytics. And I merged main, that changes the default bundle to be com.tramline.ueno so it'll show up under Ueno Android on Firebase.

@kitallis kitallis merged commit 535fc30 into main May 19, 2024
@kitallis kitallis deleted the setup-crashlytics branch May 19, 2024 14:42
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.

2 participants