always include num_courses in the event

This commit is contained in:
Gabe Mulley
2017-11-15 10:21:19 -05:00
parent 6f20cd5cf2
commit 53544a708e

View File

@@ -207,13 +207,11 @@ def _track_message_sent(site, user, msg):
'send_uuid': msg.send_uuid,
}
course_ids = msg.context.get('course_ids', [])
properties['num_courses'] = len(course_ids)
if len(course_ids) > 0:
properties['course_ids'] = course_ids[:10]
properties['primary_course_id'] = course_ids[0]
if len(course_ids) > 1:
properties['num_courses'] = len(course_ids)
analytics.track(
user_id=user.id,
event='edx.bi.email.sent',