Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #18127 from edx/McKenzieW/learner-3830

Browse Source 
swap page names for entitlement session analytics
This commit is contained in:
McKenzie Welter
2018-05-03 15:39:43 -04:00
committed by GitHub
parent eb3e8d9585 5c4c86afce
commit be097fed99
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lms/static/js/learner_dashboard/views/course_entitlement_view.js
View File

@@ -134,7 +134,7 @@ class CourseEntitlementView extends Backbone.View {
3) Hide the 'View Course' button to the course card.
*/
const successIconEl = '<span class="fa fa-check" aria-hidden="true"></span>';
const eventPage = this.$parentEl ? 'course-dashboard' : 'program-details';
const eventPage = this.$parentEl ? 'program-details' : 'course-dashboard';
const eventAction = prevSession ? 'switch' : 'new';
// Emit analytics event to track user leaving current session
@@ -178,7 +178,7 @@ class CourseEntitlementView extends Backbone.View {
4) Remove the link from the course card image and title.
*/
// Emit analytics event to track user leaving current session
const eventPage = this.$parentEl ? 'course-dashboard' : 'program-details';
const eventPage = this.$parentEl ? 'program-details' : 'course-dashboard';
this.trackSessionChange(eventPage, 'leave', prevSession);
// With a containing backbone view, we can simply re-render the parent card