diff --git a/lms/djangoapps/courseware/tests/test_submitting_problems.py b/lms/djangoapps/courseware/tests/test_submitting_problems.py index bdb0d86c4c..1eaad5f301 100644 --- a/lms/djangoapps/courseware/tests/test_submitting_problems.py +++ b/lms/djangoapps/courseware/tests/test_submitting_problems.py @@ -627,7 +627,7 @@ class TestCourseGrader(TestSubmittingProblems): "namespace": "grade", "name": "grade", "display_name": "Grade", - "criteria": {"min_grade": 0.52} + "criteria": {"min_grade": 0.52}, }] # Add a single credit requirement (final grade) set_credit_requirements(self.course.id, requirements) diff --git a/lms/djangoapps/courseware/views.py b/lms/djangoapps/courseware/views.py index de5252bbfe..d7275c7fd5 100644 --- a/lms/djangoapps/courseware/views.py +++ b/lms/djangoapps/courseware/views.py @@ -1153,14 +1153,9 @@ def _credit_course_requirements(course_key, student): else: eligibility_status = "partial_eligible" - paired_requirements = {} - for requirement in requirement_statuses: - namespace = requirement.pop("namespace") - paired_requirements.setdefault(namespace, []).append(requirement) - return { 'eligibility_status': eligibility_status, - 'requirements': OrderedDict(sorted(paired_requirements.items(), reverse=True)) + 'requirements': requirement_statuses, } diff --git a/lms/templates/courseware/progress.html b/lms/templates/courseware/progress.html index 5ceac475e1..14bf9d77fb 100644 --- a/lms/templates/courseware/progress.html +++ b/lms/templates/courseware/progress.html @@ -120,27 +120,25 @@ from django.utils.http import urlquote_plus %endif
- %for namespace in credit_course_requirements['requirements']: - %for requirement in credit_course_requirements['requirements'][namespace]: -
-
${_(requirement['display_name'])}
-
- %if requirement['status']: - %if requirement['status'] == 'submitted': - - %elif requirement['status'] == 'failed': - - ${_("Verification Failed" )} - %elif requirement['status'] == 'satisfied': - - Verified on ${get_time_display(requirement['status_date'], DEFAULT_SHORT_DATE_FORMAT, settings.TIME_ZONE)} - %endif - %else: - ${_("Upcoming")} + %for requirement in credit_course_requirements['requirements']: +
+
${_(requirement['display_name'])}
+
+ %if requirement['status']: + %if requirement['status'] == 'submitted': + + %elif requirement['status'] == 'failed': + + ${_("Verification Failed" )} + %elif requirement['status'] == 'satisfied': + + Verified on ${get_time_display(requirement['status_date'], DEFAULT_SHORT_DATE_FORMAT, settings.TIME_ZONE)} %endif -
+ %else: + ${_("Upcoming")} + %endif
- %endfor +
%endfor