Merge pull request #87 from edx/engagement-updates

Engagement updates

analytics_dashboard/courses/templates/courses/engagement_content.html

@@ -37,19 +37,19 @@ Individual course-centric engagement content view.
         {% include "summary_point.html" with count=summary.any|intcomma label="Active Students" subheading="last week" tooltip=tooltips.all_activity_summary only %}
       </div>
 
-      {% switch show_engagement_forum_activity %}
-      <div class="col-xs-12 col-sm-3" data-activity-type="posted_forum">
-        {% include "summary_point.html" with count=summary.posted_forum|intcomma label="Posted in Forum" subheading="last week" tooltip=tooltips.posted_forum_summary only %}
+      <div class="col-xs-12 col-sm-3" data-activity-type="played_video">
+        {% include "summary_point.html" with count=summary.played_video label="Watched a Video" subheading="last week" tooltip=tooltips.played_video_summary only %}
       </div>
-      {% endswitch %}
 
       <div class="col-xs-12 col-sm-3" data-activity-type="attempted_problem">
         {% include "summary_point.html" with count=summary.attempted_problem|intcomma label="Tried a Problem" subheading="last week" tooltip=tooltips.attempted_problem_summary only %}
       </div>
 
-      <div class="col-xs-12 col-sm-3" data-activity-type="played_video">
-        {% include "summary_point.html" with count=summary.played_video label="Watched a Video" subheading="last week" tooltip=tooltips.played_video_summary only %}
+      {% switch show_engagement_forum_activity %}
+      <div class="col-xs-12 col-sm-3" data-activity-type="posted_forum">
+        {% include "summary_point.html" with count=summary.posted_forum|intcomma label="Posted in Forum" subheading="last week" tooltip=tooltips.posted_forum_summary only %}
       </div>
+      {% endswitch %}
     </div>
   </div>
 

analytics_dashboard/courses/tests/test_views.py

@@ -166,11 +166,10 @@ class CourseEngagementContentViewTests(CourseViewTestMixin, TestCase):
 
         # check to make sure that we have tooltips
         self.assertDictEqual(response.context['tooltips'], {
-            'all_activity_summary': 'Students who initiated an action.',
-            # pylint: disable=line-too-long
-            'posted_forum_summary': 'Students who created a post, responded to a post, or made a comment in any discussion.',
-            'attempted_problem_summary': 'Students who answered any question.',
-            'played_video_summary': 'Students who started watching any video.'
+            'all_activity_summary': 'Students who interacted with at least one page, video, problem, or discussion',
+            'posted_forum_summary': 'Students who contributed to any discussion topic',
+            'attempted_problem_summary': 'Students who submitted a standard problem',
+            'played_video_summary': 'Students who played one or more videos'
         })
 
         # check page title

analytics_dashboard/courses/views.py

@@ -332,11 +332,11 @@ class EngagementContentView(EngagementTemplateView):
         context = super(EngagementContentView, self).get_context_data(**kwargs)
 
         tooltips = {
-            'all_activity_summary': _('Students who initiated an action.'),
+            'all_activity_summary': _('Students who interacted with at least one page, video, problem, or discussion'),
             'posted_forum_summary': _(
-                'Students who created a post, responded to a post, or made a comment in any discussion.'),
-            'attempted_problem_summary': _('Students who answered any question.'),
-            'played_video_summary': _('Students who started watching any video.'),
+                'Students who contributed to any discussion topic'),
+            'attempted_problem_summary': _('Students who submitted a standard problem'),
+            'played_video_summary': _('Students who played one or more videos'),
         }
 
         presenter = CourseEngagementPresenter()