From 74d03809a78848872a5328e9db24949eec5f3f50 Mon Sep 17 00:00:00 2001
From: Renzo Lucioni <renzolucioni@gmail.com>
Date: Thu, 3 Apr 2014 15:05:57 -0400
Subject: [PATCH 1/3] Add more granular pageview tracking, update Segment.io
 snippet

---
 lms/templates/widgets/segment-io.html | 29 ++++++++++++++++++++++-----
 1 file changed, 24 insertions(+), 5 deletions(-)

diff --git a/lms/templates/widgets/segment-io.html b/lms/templates/widgets/segment-io.html
index 3c6728bb16..e627ea2d11 100644
--- a/lms/templates/widgets/segment-io.html
+++ b/lms/templates/widgets/segment-io.html
@@ -6,15 +6,34 @@
 <% active_flags = " + ".join(waffle.get_flags(request)) %>
 
 <script type="text/javascript">
-  var analytics=analytics||[];analytics.load=function(e){var t=document.createElement("script");t.type="text/javascript",t.async=!0,t.src=("https:"===document.location.protocol?"https://":"http://")+"d2dq2ahtl5zl1z.cloudfront.net/analytics.js/v1/"+e+"/analytics.min.js";var n=document.getElementsByTagName("script")[0];n.parentNode.insertBefore(t,n);var r=function(e){return function(){analytics.push([e].concat(Array.prototype.slice.call(arguments,0)))}},i=["identify","track","trackLink","trackForm","trackClick","trackSubmit","pageview","ab","alias","ready"];for(var s=0;s<i.length;s++)analytics[i[s]]=r(i[s])};
-  analytics.load("${ settings.SEGMENT_IO_LMS_KEY }");
+  window.analytics||(window.analytics=[]),window.analytics.methods=["identify","track","trackLink","trackForm","trackClick","trackSubmit","page","pageview","ab","alias","ready","group","on","once","off"],window.analytics.factory=function(t){return function(){var a=Array.prototype.slice.call(arguments);return a.unshift(t),window.analytics.push(a),window.analytics}};for(var i=0;i<window.analytics.methods.length;i++){var method=window.analytics.methods[i];window.analytics[method]=window.analytics.factory(method)}window.analytics.load=function(t){var a=document.createElement("script");a.type="text/javascript",a.async=!0,a.src=("https:"===document.location.protocol?"https://":"http://")+"d2dq2ahtl5zl1z.cloudfront.net/analytics.js/v1/"+t+"/analytics.min.js";var n=document.getElementsByTagName("script")[0];n.parentNode.insertBefore(a,n)},window.analytics.SNIPPET_VERSION="2.0.8",
+  window.analytics.load("${ settings.SEGMENT_IO_LMS_KEY }");
+
+  path = window.location.href.split("/").slice(3)[0]
+  if (path == "") {
+    window.analytics.page("Home");
+  } else if (path == "register") {
+    window.analytics.page("Registration");
+  } else if (path == "login") {
+    window.analytics.page("Login");
+  } else if (path == "dashboard") {
+    window.analytics.page("Dashboard");
+  } else if (path == "course-list") {
+    window.analytics.page("Course Listing");
+  } else if (path == "course") {
+    window.analytics.page("Course About");
+  } else {
+    window.analytics.page("Other");
+  }
 
   % if user.is_authenticated():
 
     analytics.identify("${ user.id }", {
-      email          : "${ user.email }",
-      username       : "${ user.username }",
-      "Active Flags" : "${ active_flags }",
+      "Registered"      : true,
+      email             : "${ user.email }",
+      username          : "${ user.username }",
+      "Enrollment Count": ${ sum(1 for course in user.courseenrollment_set.values() if course['is_active'] == True) },
+      "Active Flags"    : "${ active_flags }",
     });
 
   % endif

From da511808150638795bafc08c9c779d8566bab658 Mon Sep 17 00:00:00 2001
From: Renzo Lucioni <renzolucioni@gmail.com>
Date: Thu, 10 Apr 2014 15:09:37 -0400
Subject: [PATCH 2/3] Add comments explaining analytics improvements

---
 lms/templates/widgets/segment-io.html | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/lms/templates/widgets/segment-io.html b/lms/templates/widgets/segment-io.html
index e627ea2d11..21825767a7 100644
--- a/lms/templates/widgets/segment-io.html
+++ b/lms/templates/widgets/segment-io.html
@@ -6,23 +6,33 @@
 <% active_flags = " + ".join(waffle.get_flags(request)) %>
 
 <script type="text/javascript">
+  // Asynchronously load Segment.io's analytics.js library
   window.analytics||(window.analytics=[]),window.analytics.methods=["identify","track","trackLink","trackForm","trackClick","trackSubmit","page","pageview","ab","alias","ready","group","on","once","off"],window.analytics.factory=function(t){return function(){var a=Array.prototype.slice.call(arguments);return a.unshift(t),window.analytics.push(a),window.analytics}};for(var i=0;i<window.analytics.methods.length;i++){var method=window.analytics.methods[i];window.analytics[method]=window.analytics.factory(method)}window.analytics.load=function(t){var a=document.createElement("script");a.type="text/javascript",a.async=!0,a.src=("https:"===document.location.protocol?"https://":"http://")+"d2dq2ahtl5zl1z.cloudfront.net/analytics.js/v1/"+t+"/analytics.min.js";var n=document.getElementsByTagName("script")[0];n.parentNode.insertBefore(a,n)},window.analytics.SNIPPET_VERSION="2.0.8",
   window.analytics.load("${ settings.SEGMENT_IO_LMS_KEY }");
 
-  path = window.location.href.split("/").slice(3)[0]
+  // Get current page URL and pull out the path
+  path = window.location.href.split("/")[3]
+  // Match on the current path and fire the appropriate pageview event
   if (path == "") {
+    // Home page viewed (path empty)
     window.analytics.page("Home");
   } else if (path == "register") {
+    // Registration page viewed
     window.analytics.page("Registration");
   } else if (path == "login") {
+    // Login page viewed
     window.analytics.page("Login");
   } else if (path == "dashboard") {
+    // Dashboard viewed
     window.analytics.page("Dashboard");
   } else if (path == "course-list") {
+    // Course listing page viewed
     window.analytics.page("Course Listing");
   } else if (path == "course") {
+    // Course about page viewed
     window.analytics.page("Course About");
   } else {
+    // This event serves as a catch-all, firing when any other page is viewed
     window.analytics.page("Other");
   }
 
@@ -32,6 +42,7 @@
       "Registered"      : true,
       email             : "${ user.email }",
       username          : "${ user.username }",
+      // Count the number of courses in which the user is currently enrolled
       "Enrollment Count": ${ sum(1 for course in user.courseenrollment_set.values() if course['is_active'] == True) },
       "Active Flags"    : "${ active_flags }",
     });

From a57a62807eac6386685ab9aa41f8197c0c7ef2ae Mon Sep 17 00:00:00 2001
From: Renzo Lucioni <renzolucioni@gmail.com>
Date: Mon, 14 Apr 2014 18:20:13 -0400
Subject: [PATCH 3/3] Remove page view events corresponding to Drupal pages

---
 lms/templates/widgets/segment-io.html | 11 +----------
 1 file changed, 1 insertion(+), 10 deletions(-)

diff --git a/lms/templates/widgets/segment-io.html b/lms/templates/widgets/segment-io.html
index 21825767a7..8d4133c75c 100644
--- a/lms/templates/widgets/segment-io.html
+++ b/lms/templates/widgets/segment-io.html
@@ -13,10 +13,7 @@
   // Get current page URL and pull out the path
   path = window.location.href.split("/")[3]
   // Match on the current path and fire the appropriate pageview event
-  if (path == "") {
-    // Home page viewed (path empty)
-    window.analytics.page("Home");
-  } else if (path == "register") {
+  if (path == "register") {
     // Registration page viewed
     window.analytics.page("Registration");
   } else if (path == "login") {
@@ -25,12 +22,6 @@
   } else if (path == "dashboard") {
     // Dashboard viewed
     window.analytics.page("Dashboard");
-  } else if (path == "course-list") {
-    // Course listing page viewed
-    window.analytics.page("Course Listing");
-  } else if (path == "course") {
-    // Course about page viewed
-    window.analytics.page("Course About");
   } else {
     // This event serves as a catch-all, firing when any other page is viewed
     window.analytics.page("Other");