Merge pull request #1629 from edx/adam/fix/enrollment-count-intr-dash
makes beta dash count active users in enrollment count [AN-115]
This commit is contained in:
@@ -5,6 +5,9 @@ These are notable changes in edx-platform. This is a rolling list of changes,
|
||||
in roughly chronological order, most recent first. Add your entries at or near
|
||||
the top. Include a label indicating the component affected.
|
||||
|
||||
LMS: Beta instructor dashboard will only count actively enrolled students for
|
||||
course enrollment numbers.
|
||||
|
||||
LMS: Users with is_staff=True no longer have the STAFF label appear on
|
||||
their forum posts.
|
||||
|
||||
|
||||
@@ -108,7 +108,7 @@ def _section_course_info(course_id, access):
|
||||
'course_num': course_num,
|
||||
'course_name': course_name,
|
||||
'course_display_name': course.display_name,
|
||||
'enrollment_count': CourseEnrollment.objects.filter(course_id=course_id).count(),
|
||||
'enrollment_count': CourseEnrollment.objects.filter(course_id=course_id, is_active=1).count(),
|
||||
'has_started': course.has_started(),
|
||||
'has_ended': course.has_ended(),
|
||||
'list_instructor_tasks_url': reverse('list_instructor_tasks', kwargs={'course_id': course_id}),
|
||||
|
||||
Reference in New Issue
Block a user