Skip to content
This repository has been archived by the owner on Nov 4, 2024. It is now read-only.

Commit

Permalink
Disabled client-side tracking if Segment ID is empty
Browse files Browse the repository at this point in the history
Tracking is now disabled if the Segment application ID is set to an empty string (e.g. the default for local development).

LEARNER-1668
  • Loading branch information
clintonb committed Jul 6, 2017
1 parent f0035e3 commit 1a38a36
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ecommerce/static/js/models/tracking_model.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ define(['backbone', 'underscore'], function(Backbone, _) {
isTracking: function() {
var self = this,
trackId = self.get('segmentApplicationId');
return !_(trackId).isUndefined() && !_(trackId).isNull();
return !_(trackId).isUndefined() && !_(trackId).isNull() && !_(trackId).isEmpty();
}
});
});

0 comments on commit 1a38a36

Please sign in to comment.