Merge pull request #175 from Microsoft/disable-metrics-in-debug

Disable metrics reporting in debug mode

Author: geof90

CodePush.m

@@ -512,6 +512,15 @@ - (void)savePendingUpdate:(NSString *)packageHash
 RCT_EXPORT_METHOD(getNewStatusReport:(RCTPromiseResolveBlock)resolve
                             rejecter:(RCTPromiseRejectBlock)reject)
 {
+#ifdef DEBUG
+    // Do not report metrics if running in debug mode.
+    resolve(nil);
+#else
+    if ([_bridge.bundleURL.scheme hasPrefix:@"http"]) {
+        // Do not report metrics if running bundle from packager.
+        resolve(nil);
+        return;
+    }
 
     dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^{
         if (needToReportRollback) {
@@ -532,15 +541,15 @@ - (void)savePendingUpdate:(NSString *)packageHash
                 resolve([CodePushTelemetryManager getUpdateReport:currentPackage]);
                 return;
             }
-        } else if (isRunningBinaryVersion || [_bridge.bundleURL.scheme hasPrefix:@"http"]) {
-            // Check if the current appVersion has been reported.
+        } else if (isRunningBinaryVersion) {
             NSString *appVersion = [[CodePushConfig current] appVersion];
             resolve([CodePushTelemetryManager getBinaryUpdateReport:appVersion]);
             return;
         }
 
         resolve(nil);
     });
+#endif
 }
 
 /*

android/app/build.gradle

@@ -22,7 +22,7 @@ android {
 }
 
 dependencies {
-    compile fileTree(dir: 'libs', include: ['*.jar'])
-    compile 'com.android.support:appcompat-v7:23.0.0'
-    compile 'com.facebook.react:react-native:0.15.1'
+    compile fileTree(dir: "libs", include: ["*.jar"])
+    compile "com.android.support:appcompat-v7:23.0.1"
+    compile "com.facebook.react:react-native:0.19.+"
 }

android/app/src/main/java/com/microsoft/codepush/react/CodePush.java

@@ -426,6 +426,11 @@ protected Void doInBackground(Object... params) {
 
         @ReactMethod
         public void getNewStatusReport(final Promise promise) {
+            if (isDebugMode) {
+                // Do not report metrics if running in debug mode.
+                promise.resolve("");
+                return;
+            }
 
             AsyncTask asyncTask = new AsyncTask() {
                 @Override