Commit 4d5f5c3f by Gregory Martin

Add Forum Notifications Email Tracking

parent 82ad9295
...@@ -5,6 +5,7 @@ pertaining to new discussion forum comments. ...@@ -5,6 +5,7 @@ pertaining to new discussion forum comments.
import logging import logging
from urlparse import urljoin from urlparse import urljoin
import analytics
from celery import task from celery import task
from django.conf import settings from django.conf import settings
from django.contrib.auth.models import User from django.contrib.auth.models import User
...@@ -51,6 +52,29 @@ def send_ace_message(context): ...@@ -51,6 +52,29 @@ def send_ace_message(context):
) )
log.info('Sending forum comment email notification with context %s', message_context) log.info('Sending forum comment email notification with context %s', message_context)
ace.send(message) ace.send(message)
_track_notification_sent(message, context)
def _track_notification_sent(message, context):
"""
Send analytics event for a sent email
"""
properties = {
'site': context['site'].domain,
'app_label': 'discussion',
'name': 'responsenotification',
'language': message.language,
'uuid': unicode(message.uuid),
'send_uuid': unicode(message.send_uuid),
'thread_id': context['thread_id'],
'thread_created_at': date.deserialize(context['thread_created_at'])
}
analytics.track(
user_id=context['thread_author_id'],
event='edx.bi.email.sent',
properties=properties,
course_id=context['course_id']
)
def _should_send_message(context): def _should_send_message(context):
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment