Rename Scaled grading type to Proportional (#6843)

hypothesis · Nov 7, 2024 · ebbcca9 · ebbcca9
1 parent d93be92
commit ebbcca9
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 10 deletions.
diff --git a/lms/static/scripts/frontend_apps/api-types.ts b/lms/static/scripts/frontend_apps/api-types.ts
@@ -226,6 +226,9 @@ type AssignmentWithCourse = Assignment & {
  * - scaled: students may get a proportional grade based on the amount of
  *           annotations. If requirement is 4, and they created 3, they'll
  *           get a 75%
+ *
+ * @todo `scaled` is now referenced as `proportional` in user-facing texts.
+ *       We should consolidate it eventually.
  */
 export type GradingType = 'all_or_nothing' | 'scaled';
 

diff --git a/lms/static/scripts/frontend_apps/components/AutoGradingConfigurator.tsx b/lms/static/scripts/frontend_apps/components/AutoGradingConfigurator.tsx
@@ -122,16 +122,16 @@ export default function AutoGradingConfigurator({
               }
             >
               <RadioGroup.Radio
-                value="all_or_nothing"
-                subtitle={<small>Must meet minimum requirements.</small>}
+                value="scaled"
+                subtitle={<small>3 annotations out of 4 is 75%</small>}
               >
-                All or nothing
+                Proportional
               </RadioGroup.Radio>
               <RadioGroup.Radio
-                value="scaled"
-                subtitle={<small>Proportional to percent completed.</small>}
+                value="all_or_nothing"
+                subtitle={<small>3 annotations out of 4 is 0%</small>}
               >
-                Scaled
+                All or nothing
               </RadioGroup.Radio>
             </RadioGroup>
           </div>

diff --git a/lms/static/scripts/frontend_apps/components/FilePickerApp.tsx b/lms/static/scripts/frontend_apps/components/FilePickerApp.tsx
@@ -203,7 +203,7 @@ export default function FilePickerApp({ onSubmit }: FilePickerAppProps) {
       if (!assignmentAutoGradingConfig) {
         return {
           enabled: false,
-          grading_type: 'all_or_nothing',
+          grading_type: 'scaled',
           activity_calculation: 'cumulative',
           required_annotations: 1,
         };

diff --git a/lms/static/scripts/frontend_apps/components/dashboard/test/GradeIndicator-test.js b/lms/static/scripts/frontend_apps/components/dashboard/test/GradeIndicator-test.js
@@ -6,7 +6,7 @@ import GradeIndicator from '../GradeIndicator';
 
 describe('GradeIndicator', () => {
   const defaultConfig = {
-    grading_type: 'all_or_nothing',
+    grading_type: 'scaled',
     activity_calculation: 'separate',
     required_annotations: 1,
     required_replies: 1,

diff --git a/lms/static/scripts/frontend_apps/components/test/AutoGradingConfigurator-test.js b/lms/static/scripts/frontend_apps/components/test/AutoGradingConfigurator-test.js
@@ -10,7 +10,7 @@ describe('AutoGradingConfigurator', () => {
 
   beforeEach(() => {
     fakeAutoGradingConfig = {
-      grading_type: 'all_or_nothing',
+      grading_type: 'scaled',
       activity_calculation: 'cumulative',
       required_annotations: 1,
     };
@@ -80,7 +80,7 @@ describe('AutoGradingConfigurator', () => {
         assert.equal(inputs.length, activityCalculation === 'separate' ? 2 : 1);
         assert.equal(
           firstInput.text(),
-          `Annotations${activityCalculation === 'cumulative' ? ' and replies' : ''}Minimum`,
+          `Annotations${activityCalculation === 'cumulative' ? ' and replies' : ''}Goal`,
         );
       });
     });