Remove Fabric and Crashlytics

The Logger now logs to `console.log` exclusively.

.android.env.example

@@ -1,5 +1,4 @@
 export MM_FOX_CODE=
-export MM_FABRIC_API_KEY=
 export MM_BRANCH_KEY_TEST=
 export MM_BRANCH_KEY_LIVE=
 export MM_MIXPANEL_TOKEN=

.gitignore

@@ -29,8 +29,6 @@ app/bin
 .gradle
 local.properties
 *.iml
-android/app/src/main/assets/crashlytics-build.properties
-android/app/src/main/res/values/com_crashlytics_export_strings.xml
 android/.project
 android/app/.project
 android/app/bin/

.ios.env.example

@@ -1,4 +1,3 @@
 MM_FOX_CODE =
-MM_FABRIC_API_KEY =
 MM_BRANCH_KEY_TEST =
 MM_BRANCH_KEY_LIVE =

android/app/build.gradle

@@ -24,31 +24,8 @@ def getPassword(String currentUser, String keyChain) {
     stdout.toString().trim()
 }
 
-buildscript {
-    repositories {
-        maven { url 'https://maven.fabric.io/public' }
-    }
-
-    dependencies {
-        // These docs use an open ended version so that our plugin
-        // can be updated quickly in response to Android tooling updates
-
-        // We recommend changing it to the latest version from our changelog:
-        // https://docs.fabric.io/android/changelog.html#fabric-gradle-plugin
-        classpath 'io.fabric.tools:gradle:1.+'
-    }
-}
-
 apply plugin: "com.android.application"
 
-apply plugin: 'io.fabric'
-
-
-repositories {
-    maven { url 'https://maven.fabric.io/public' }
-	jcenter()
-}
-
 import com.android.build.OutputFile
 
 /**
@@ -142,27 +119,8 @@ def enableSeparateBuildPerCPUArchitecture = false
  */
 def enableProguardInReleaseBuilds = false
 
-/**
-*
-* override fabric properties file if MM_FABRIC_API_KEY is set
-*/
-def buildFabricPropertiesIfNeeded() {
-    def FABRIC_API_KEY = System.getenv('MM_FABRIC_API_KEY')
-    if (FABRIC_API_KEY) {
-        def commentMessage = "AUTOGEN FABRIC PROPERTIES"
-        ant.propertyfile(file: "fabric.properties", comment: commentMessage) {
-            entry(key: "apiKey", value: FABRIC_API_KEY)
-        }
-    }
-}
-
 android {
 
-
-	afterEvaluate {
-		buildFabricPropertiesIfNeeded()
-	}
-
     compileSdkVersion rootProject.ext.compileSdkVersion
 
 	compileOptions {
@@ -253,7 +211,6 @@ dependencies {
     implementation project(':react-native-sensors')
     implementation project(':react-native-reanimated')
     implementation project(':react-native-webview')
-    implementation project(':react-native-fabric')
     implementation project(':@react-native-community_netinfo')
     implementation project(':react-native-view-shot')
     implementation project(':lottie-react-native')
@@ -285,9 +242,6 @@ dependencies {
     implementation project(':react-native-vector-icons')
 	implementation 'com.mixpanel.android:mixpanel-android:5.+'
 
-    implementation('com.crashlytics.sdk.android:crashlytics:2.9.4@aar') {
-        transitive = true;
-    }
     androidTestImplementation('com.wix:detox:+') { transitive = true }
     androidTestImplementation 'junit:junit:4.12'
 }

android/app/src/main/java/io/metamask/MainApplication.java

@@ -4,7 +4,6 @@
 import com.sensors.RNSensorsPackage;
 import com.swmansion.reanimated.ReanimatedPackage;
 import com.reactnativecommunity.webview.RNCWebViewPackage;
-import com.smixx.fabric.FabricPackage;
 import com.reactnativecommunity.netinfo.NetInfoPackage;
 import fr.greweb.reactnativeviewshot.RNViewShotPackage;
 import com.airbnb.android.react.lottie.LottiePackage;
@@ -53,7 +52,6 @@ protected List<ReactPackage> getPackages() {
             	new RNSensorsPackage(),
             	new ReanimatedPackage(),
 				new RNCWebViewPackage(),
-            	new FabricPackage(),
 				new NetInfoPackage(),
             	new RNViewShotPackage(),
             	new LottiePackage(),

android/app/src/main/java/io/metamask/nativeModules/RCTAnalytics.java

@@ -4,8 +4,6 @@
 import android.content.pm.PackageManager;
 import android.util.Log;
 
-import com.crashlytics.android.Crashlytics;
-import com.crashlytics.android.core.CrashlyticsCore;
 import com.facebook.react.bridge.ReactApplicationContext;
 import com.facebook.react.bridge.ReactContextBaseJavaModule;
 import com.facebook.react.bridge.ReactMethod;
@@ -23,8 +21,6 @@
 import java.util.ArrayList;
 import java.util.HashMap;
 
-import io.fabric.sdk.android.Fabric;
-
 public class RCTAnalytics extends ReactContextBaseJavaModule {
 
 	MixpanelAPI mixpanel;
@@ -63,7 +59,6 @@ public void optIn(boolean val) {
 
         if(val){
 			this.mixpanel.optInTracking();
-            Fabric.with(this.getReactApplicationContext(), new Crashlytics());
 
 		}else{
 			this.mixpanel.optOutTracking();

android/build.gradle

@@ -57,17 +57,3 @@ allprojects {
         }
     }
 }
-
-subprojects {project ->
-    if (project.name.contains('react-native-fabric')) {
-        buildscript {
-            repositories {
-				google()
-				jcenter()
-                maven {
-					   url = 'https://dl.bintray.com/android/android-tools/'
-				}
-            }
-        }
-    }
-}

android/settings.gradle

@@ -5,8 +5,6 @@ include ':react-native-reanimated'
 project(':react-native-reanimated').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-reanimated/android')
 include ':react-native-webview'
 project(':react-native-webview').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-webview/android')
-include ':react-native-fabric'
-project(':react-native-fabric').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-fabric/android')
 include ':@react-native-community_netinfo'
 project(':@react-native-community_netinfo').projectDir = new File(rootProject.projectDir, '../node_modules/@react-native-community/netinfo/android')
 include ':react-native-view-shot'

app/util/Logger.js

@@ -1,7 +1,4 @@
 'use strict';
-// eslint-disable-next-line import/default
-import Fabric from 'react-native-fabric';
-import { Platform } from 'react-native';
 import AsyncStorage from '@react-native-community/async-storage';
 
 /**
@@ -18,14 +15,13 @@ export default class Logger {
 	 * @returns - void
 	 */
 	static async log(...args) {
-		// TODO use crashlytics opt-in
 		// Check if user passed accepted opt-in to metrics
 		const metricsOptIn = await AsyncStorage.getItem('@MetaMask:metricsOptIn');
 		if (__DEV__) {
 			args.unshift('[MetaMask DEBUG]:');
 			console.log.apply(null, args); // eslint-disable-line no-console
 		} else if (metricsOptIn === 'agreed') {
-			Fabric.Crashlytics.log(JSON.stringify(args));
+			console.log(args);
 		}
 	}
 
@@ -36,18 +32,13 @@ export default class Logger {
 	 * @returns - void
 	 */
 	static async error(...args) {
-		// TODO use crashlytics opt-in
 		// Check if user passed accepted opt-in to metrics
 		const metricsOptIn = await AsyncStorage.getItem('@MetaMask:metricsOptIn');
 		if (__DEV__) {
 			args.unshift('[MetaMask DEBUG]:');
 			console.warn(args); // eslint-disable-line no-console
 		} else if (metricsOptIn === 'agreed') {
-			if (Platform.OS === 'android') {
-				Fabric.Crashlytics.logException(JSON.stringify(args));
-			} else {
-				Fabric.Crashlytics.recordError(JSON.stringify(args));
-			}
+			console.log(args);
 		}
 	}
 }

app/util/testSetup.js

@@ -82,7 +82,6 @@ jest.mock('../core/Engine', () => ({
 
 jest.mock('react-native-keychain', () => ({ getSupportedBiometryType: () => Promise.resolve('FaceId') }));
 jest.mock('react-native-share', () => 'RNShare');
-jest.mock('react-native-fabric', () => 'Fabric');
 jest.mock('react-native-branch', () => 'RNBranch');
 jest.mock('react-native-sensors', () => 'RNSensors');
 jest.mock('react-native-device-info', () => 'DeviceInfo');