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

Merge pull request #5073 from edx/ecom/fix-user-id

Browse Source 
Analytics should use user.id not anon id
This commit is contained in:
Julia Hansbrough
2014-09-04 10:00:28 -04:00
parent a8d07780fc 0dea82e43e
commit 23d3a9187c
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
common/djangoapps/student/views.py
View File

@@ -986,7 +986,7 @@ def login_user(request, error=""): # pylint: disable-msg=too-many-statements,un
# Track the user's sign in
if settings.FEATURES.get('SEGMENT_IO_LMS') and hasattr(settings, 'SEGMENT_IO_LMS_KEY'):
tracking_context = tracker.get_tracker().resolve_context()
analytics.identify(anonymous_id_for_user(user, None), {
analytics.identify(user.id, {
'email': email,
'username': username,
})
@@ -1461,7 +1461,7 @@ def create_account(request, post_override=None): # pylint: disable-msg=too-many
# Track the user's registration
if settings.FEATURES.get('SEGMENT_IO_LMS') and hasattr(settings, 'SEGMENT_IO_LMS_KEY'):
tracking_context = tracker.get_tracker().resolve_context()
analytics.identify(anonymous_id_for_user(user, None), {
analytics.identify(user.id, {
email: email,
username: username,
})