Merge pull request #253 from hello/dz/more_segment_fixes

Fix for the new segment crash on upgrade. [ref #109196952]
hello · Nov 30, 2015 · e52b470 · e52b470
2 parents fa8db9d + 33d4e1e
commit e52b470
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/app/src/main/java/is/hello/sense/util/Analytics.java b/app/src/main/java/is/hello/sense/util/Analytics.java
@@ -532,10 +532,17 @@ public static void trackRegistration(@Nullable String accountId, @Nullable final
         trackUserIdentifier(accountId);
     }
 
-    public static void trackSignIn(@NonNull final String accountId, @Nullable final String name, @Nullable final String email) {
+    public static void trackSignIn(@NonNull final String accountId, @Nullable  String name, @Nullable  String email) {
         if (context == null){
             return;
         }
+        if (name == null) {
+            name = "";
+        }
+        if (email == null) {
+            email = "";
+        }
+
         com.segment.analytics.Analytics.with(context).identify(accountId);
         Analytics.trackEvent(Analytics.Global.EVENT_SIGNED_IN, null);