feat: added user email in email sent event data (openedx#35456)

edly-io · Sep 16, 2024 · 1103544 · 1103544
1 parent bb6ac5a
commit 1103544
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/lms/djangoapps/bulk_email/signals.py b/lms/djangoapps/bulk_email/signals.py
@@ -55,5 +55,6 @@ def ace_email_sent_handler(sender, **kwargs):
             'channel': channel,
             'course_id': course_id,
             'user_id': user_id,
+            'user_email': message.recipient.email_address,
         }
     )
diff --git a/lms/djangoapps/bulk_email/tasks.py b/lms/djangoapps/bulk_email/tasks.py
@@ -533,7 +533,7 @@ def _send_course_email(entry_id, email_id, to_list, global_email_context, subtas
             email_context['email'] = email
             email_context['name'] = profile_name
             email_context['user_id'] = user_id
-            email_context['course_id'] = course_email.course_id
+            email_context['course_id'] = str(course_email.course_id)
             email_context['unsubscribe_link'] = get_unsubscribed_link(current_recipient['username'],
                                                                       str(course_email.course_id))
 

diff --git a/lms/djangoapps/bulk_email/views.py b/lms/djangoapps/bulk_email/views.py
@@ -64,6 +64,7 @@ def opt_out_email_updates(request, token, course_id):
     event_name = 'edx.bulk_email.opt_out'
     event_data = {
         "username": user.username,
+        "user_email": user.email,
         "user_id": user.id,
         "course_id": course_id,
     }