feat: Add subsection grading policy mismatch validation

Adds validation for the grading assignment and grading policy, showing a warning in Studio if there is a mismatch.
openedx · Jul 27, 2023 · 3d9e6d2 · 3d9e6d2
1 parent 3557799
commit 3d9e6d2
Showing 1 changed file with 25 additions and 3 deletions.
diff --git a/cms/templates/js/course-outline.underscore b/cms/templates/js/course-outline.underscore
@@ -78,8 +78,14 @@ if (staffOnlyMessage) {
 }
 
 var gradingType = gettext('Ungraded');
+var gradingPolicyMismatch = false;
 if (xblockInfo.get('graded')) {
     gradingType = xblockInfo.get('format')
+    if (gradingType) {
+        gradingPolicyMismatch = (
+            xblockInfo.get('course_graders').filter((cg) => cg.toLowerCase() === gradingType.toLowerCase())
+        ).length === 0;
+    }
 }
 
 var is_proctored_exam = xblockInfo.get('is_proctored_exam');
@@ -269,7 +275,7 @@ if (is_proctored_exam) {
                                     )
                                 %>
                             </span>
-                        <p>
+                        </p>
                     </div>
                 <% } %>
             <% } else if ((xblockInfo.get('due_date') && !course.get('self_paced')) || xblockInfo.get('graded')) { %>
@@ -299,7 +305,7 @@ if (is_proctored_exam) {
                                     )
                                 %>
                             </span>
-                        <p>
+                        </p>
                     </div>
                 <% } %>
             <% } else if (course.get('self_paced') && course.get('is_custom_relative_dates_active') && xblockInfo.get('relative_weeks_due')) { %>
@@ -325,7 +331,7 @@ if (is_proctored_exam) {
                                 )
                             %>
                         </span>
-                    <p>
+                    </p>
                 </div>
             <% } %>
             <div class="status-hide-after-due">
@@ -352,6 +358,22 @@ if (is_proctored_exam) {
             <% } %>
             </div>
         <% } %>
+        <% if (gradingPolicyMismatch) { %>
+            <div class="status-messages">
+                <div class="container-message wrapper-message">
+                    <div class="message has-warnings">
+                        <p class="warning">
+                            <span class="icon fa fa-warning" aria-hidden="true"></span>
+                            <%- interpolate(
+                                gettext("This subsection is configured as \"%(gradingType)s\", which doesn't exist in the current grading policy."),
+                                { gradingType: gradingType },
+                                true
+                            ) %>
+                        </p>
+                    </div>
+                </div>
+            </div>
+        <% } %>
     </div>
     <% } %>
 <% } %>