-
Notifications
You must be signed in to change notification settings - Fork 15
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
Crashes reporting #78
Comments
|
Release build must spawn email app with pasted crash log and our e-mail as the recipient. |
Hi, can I ask one question regarding crash reporting? As far as I understand from the requirements, are you good with using Firebase Crashlytics? The more app has users, the more emails need to be sent. I think solving this problem in a clever way would be with a service that provides crash reporting. I don't know maybe you have some concerns regarding privacy but I recommend firebase for this issue. |
@salihyalcin We have considered Firebase Crashlytics for this, but yes, here is the privacy concern. The app's purpose is to work with local files mostly (although remote resources might be added later). Hence it's possible to have zero internet connections for the app's work and this is security bonus. I think also that to send e-mail is the easiest sane way to solve the problem. Also the app will be checked later to be able to work without Google Services and I guess they are necessary for Crashlytics. |
Also a button "send current log" would be useful for debugging purposes. Sometimes it's really annoying to connect cable. |
Crashes should be reported automatically, this will save time for extracting logs.
This setting must be configurable in settings menu (#77) and be enabled by default for debug builds and disabled for production.
The text was updated successfully, but these errors were encountered: