shortened view_map_icons firebase event name (#118)

DerrickLi · rustielin · commit d5602236dcda · 2019-01-30T12:35:35.000-08:00
* added firebase remote config and spotlight for ab testing

* adding remote config defaults

* added firebase remote config and spotlight for ab testing

* adding remote config defaults

* AB testing spotlight should be working

* updated first session definition and added comments

* updated first session definition and added comments

* shortened view_map_icons event name for firebase
diff --git a/app/src/main/java/com/asuc/asucmobile/fragments/MapsFragment.java b/app/src/main/java/com/asuc/asucmobile/fragments/MapsFragment.java
@@ -255,7 +255,8 @@ public void onClick(View v) {
 
                 // Firebase AB test tracking, log clicks after showing spotlight once
                 if (viewedFirstSession()) {
-                    mFirebaseAnalytics.logEvent("view_map_icons_clicked_after_first_session", bundle);
+                    // Note: Firebase events must be under 40 characters in length
+                    mFirebaseAnalytics.logEvent("view_map_icons_after_first_session", bundle);
                 }
 
                 if(FABmenu.isOpened()){

Original file line number	Diff line number	Diff line change
`@@ -255,7 +255,8 @@ public void onClick(View v) {`
`255`	`255`
`256`	`256`	`// Firebase AB test tracking, log clicks after showing spotlight once`
`257`	`257`	`if (viewedFirstSession()) {`
`258`		`- mFirebaseAnalytics.logEvent("view_map_icons_clicked_after_first_session", bundle);`
	`258`	`+ // Note: Firebase events must be under 40 characters in length`
	`259`	`+ mFirebaseAnalytics.logEvent("view_map_icons_after_first_session", bundle);`
`259`	`260`	`}`
`260`	`261`
`261`	`262`	`if(FABmenu.isOpened()){`