CrashReporter icon indicating copy to clipboard operation
CrashReporter copied to clipboard

Is there any no-op implementation already?

Open mochadwi opened this issue 4 years ago • 4 comments

Does this library wouldn't be crashed if we only define the debugImplementation? The initialization class will be red in releaseImplementation

mochadwi avatar Apr 16 '20 20:04 mochadwi

cc @amitshekhariitbhu @balsikandar @janishar

mochadwi avatar Apr 16 '20 20:04 mochadwi

I have been making quite some changes to a fork of this library here as a part of it, thanks to your issue, I implemented no-op.

Please note, the readme is still to be updated and maybe a wiki in future. But, you can use

'com.orpheusdroid.crashreporter:crashreporter:1.0.3' for full implementation and 'com.orpheusdroid.crashreporter:crashreporter-no-op:1.0.3' for the no-op implementation.

Further, while using no-op make sure you wrap CrashReporterActivity with BuildConfig.debug condition if you are using it. Since the no-op version's CrashReporterActivity is an empty class while the actual one is an activity.

Refer to this example if still not clear.

vijai1996 avatar Apr 20 '20 15:04 vijai1996

will leave this issue open for future reader

mochadwi avatar Apr 27 '20 08:04 mochadwi

I have been making quite some changes to a fork of this library here as a part of it, thanks to your issue, I implemented no-op.

Please note, the readme is still to be updated and maybe a wiki in future. But, you can use

'com.orpheusdroid.crashreporter:crashreporter:1.0.3' for full implementation and 'com.orpheusdroid.crashreporter:crashreporter-no-op:1.0.3' for the no-op implementation.

Further, while using no-op make sure you wrap CrashReporterActivity with BuildConfig.debug condition if you are using it. Since the no-op version's CrashReporterActivity is an empty class while the actual one is an activity.

Refer to this example if still not clear.

thank you so much @vijai1996

mochadwi avatar Apr 27 '20 08:04 mochadwi