Merge pull request #538 from openedx/saleem-latif/ENT-9704-updates

ENT-9704: Updated text for null emails record of leaderboard.
openedx · Nov 14, 2024 · 8a50701 · 8a50701
2 parents 8016bc3 + 12d0b10
commit 8a50701
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -16,6 +16,10 @@ Unreleased
 
 =========================
 
+[10.4.0] - 2024-11-14
+---------------------
+  * Updated text for null emails record of leaderboard.
+
 [10.3.0] - 2024-11-13
 ---------------------
   * Re-write top 10 charts queries for Enrollments, Engagements and Completions

diff --git a/enterprise_data/__init__.py b/enterprise_data/__init__.py
@@ -2,4 +2,4 @@
 Enterprise data api application. This Django app exposes API endpoints used by enterprises.
 """
 
-__version__ = "10.3.0"
+__version__ = "10.4.0"
diff --git a/enterprise_data/admin_analytics/database/tables/fact_engagement_admin_dash.py b/enterprise_data/admin_analytics/database/tables/fact_engagement_admin_dash.py
@@ -11,6 +11,8 @@
 from ..utils import run_query
 from .base import BaseTable
 
+NULL_EMAIL_TEXT = 'learners who have not shared consent'
+
 
 class FactEngagementAdminDashTable(BaseTable):
     """
@@ -327,8 +329,9 @@ def get_all_leaderboard_data(
         if include_null_email:
             engagement_data_dict['None'] = find_first(engagement_data, lambda x: x['email'] is None) or {}
             completion = find_first(completion_data, lambda x: x['email'] is None) or \
-                {'course_completion_count': 'Unknown'}
+                {'course_completion_count': ''}
             engagement_data_dict['None']['course_completion_count'] = completion['course_completion_count']
+            engagement_data_dict['None']['email'] = NULL_EMAIL_TEXT
 
         return list(engagement_data_dict.values())