From 0f045ab5dbbb724fe1fd96b9d6458d98a67a10a2 Mon Sep 17 00:00:00 2001
From: Douglas Hall <dhall@edx.org>
Date: Tue, 25 Jun 2019 16:36:59 -0400
Subject: [PATCH] Add analytics tracking to Discover New header link. (#20896)

---
 lms/static/js/dashboard/track_events.js                     | 6 +++++-
 lms/templates/header/navbar-authenticated.html              | 2 +-
 .../edx.org/lms/templates/header/navbar-authenticated.html  | 2 +-
 3 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/lms/static/js/dashboard/track_events.js b/lms/static/js/dashboard/track_events.js
index 1a30196e28..9e054fe06b 100644
--- a/lms/static/js/dashboard/track_events.js
+++ b/lms/static/js/dashboard/track_events.js
@@ -73,7 +73,7 @@ var edx = edx || {};
 
     // Emit an event  when the 'Find Courses' button is clicked.
     edx.dashboard.trackFindCourseBtnClicked = function($findCoursesBtn, properties) {
-        var trackProperty = properties || {category: 'dashboard', label: null};
+        var trackProperty = properties || {category: 'dashboard', label: 'sidebar'};
         window.analytics.trackLink(
             $findCoursesBtn,
             'edx.bi.dashboard.find_courses_button.clicked',
@@ -91,5 +91,9 @@ var edx = edx || {};
         edx.dashboard.trackCourseOptionDropdownClicked($('.wrapper-action-more'));
         edx.dashboard.trackLearnVerifiedLinkClicked($('.verified-info'));
         edx.dashboard.trackFindCourseBtnClicked($('.btn-find-courses'));
+        edx.dashboard.trackFindCourseBtnClicked(
+            $('.discover-new-link'),
+            {category: 'dashboard', label: 'header'}
+        );
     });
 }(jQuery));
diff --git a/lms/templates/header/navbar-authenticated.html b/lms/templates/header/navbar-authenticated.html
index 4e32f807a7..dc0cc469d8 100644
--- a/lms/templates/header/navbar-authenticated.html
+++ b/lms/templates/header/navbar-authenticated.html
@@ -54,7 +54,7 @@ from openedx.core.djangoapps.site_configuration import helpers as configuration_
     % endif
     % if show_explore_courses:
       <div class="mobile-nav-item hidden-mobile nav-item nav-tab">
-          <a class="tab-nav-link" href="${marketing_link('COURSES')}"
+          <a class="tab-nav-link discover-new-link" href="${marketing_link('COURSES')}"
              aria-current="${'page' if '/courses' in request.path else 'false'}">
              ${_('Discover New')}
           </a>
diff --git a/themes/edx.org/lms/templates/header/navbar-authenticated.html b/themes/edx.org/lms/templates/header/navbar-authenticated.html
index ae218ec67c..ab199f32e8 100644
--- a/themes/edx.org/lms/templates/header/navbar-authenticated.html
+++ b/themes/edx.org/lms/templates/header/navbar-authenticated.html
@@ -55,7 +55,7 @@ from openedx.core.djangoapps.site_configuration import helpers as configuration_
         % endif
         % if show_explore_courses:
           <div class="mobile-nav-item hidden-mobile nav-item nav-tab">
-              <a class="tab-nav-link" href="${marketing_link('COURSES')}"
+              <a class="tab-nav-link discover-new-link" href="${marketing_link('COURSES')}"
                 aria-current="${'page' if '/courses' in request.path else 'false'}">
                 ${_('Discover New')}
               </a>