refactor: rename to optional completion

open-craft · Mar 9, 2024 · 6d3359f · 6d3359f
1 parent f21c855
commit 6d3359f
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 8 deletions.
diff --git a/src/course-home/data/api.js b/src/course-home/data/api.js
@@ -136,7 +136,7 @@ export function normalizeOutlineBlocks(courseId, blocks) {
           title: block.display_name,
           resumeBlock: block.resume_block,
           sequenceIds: block.children || [],
-          optional: block.optional_content,
+          optional: block.optional_completion,
         };
         break;
 
@@ -153,7 +153,7 @@ export function normalizeOutlineBlocks(courseId, blocks) {
           // link in the outline (even though we ignore the given url and use an internal <Link> to ourselves).
           showLink: !!block.lms_web_url,
           title: block.display_name,
-          optional: block.optional_content,
+          optional: block.optional_completion,
         };
         break;
 

diff --git a/src/course-home/outline-tab/Section.jsx b/src/course-home/outline-tab/Section.jsx
@@ -75,7 +75,7 @@ const Section = ({
       </div>
       <div className="col-10 ml-3 p-0 font-weight-bold text-dark-500">
         <span className="align-middle">{title}</span>
-        <Badge className="ml-2" variant="light" hidden={!optional}>{intl.formatMessage(messages.optionalContent)}</Badge>
+        <Badge className="ml-2" variant="light" hidden={!optional}>{intl.formatMessage(messages.optionalCompletion)}</Badge>
         <span className="sr-only">
           , {intl.formatMessage(complete ? messages.completedSection : messages.incompleteSection)}
         </span>

diff --git a/src/course-home/outline-tab/SequenceLink.jsx b/src/course-home/outline-tab/SequenceLink.jsx
@@ -133,7 +133,7 @@ const SequenceLink = ({
         </div>
         <div className="row w-100 m-0 ml-3 pl-3">
           <small className="text-body pl-2 pr-0">
-            {optional ? intl.formatMessage(messages.optionalContent) : ''}
+            {optional ? intl.formatMessage(messages.optionalCompletion) : ''}
           </small>
           <small className="text-body pl-2">
             {due ? dueDateMessage : noDueDateMessage}

diff --git a/src/course-home/outline-tab/messages.js b/src/course-home/outline-tab/messages.js
@@ -99,7 +99,7 @@ const messages = defineMessages({
     defaultMessage: 'Open',
     description: 'A button to open the given section of the course outline',
   },
-  optionalContent: {
+  optionalCompletion: {
     id: 'learning.outline.optionalBlock',
     defaultMessage: 'Optional',
     description: 'Used as a label to indicate that a section, sequence, or unit is optional.',

diff --git a/src/courseware/course/sequence/Unit.jsx b/src/courseware/course/sequence/Unit.jsx
@@ -143,7 +143,7 @@ const Unit = ({
   return (
     <div className="unit">
       <h1 className="mb-0 h3">{unit.title}</h1>
-      <Badge className="ml-2" variant="light" hidden={!unit.optional}>{intl.formatMessage(messages.optionalContent)}</Badge>
+      <Badge className="ml-2" variant="light" hidden={!unit.optional}>{intl.formatMessage(messages.optionalCompletion)}</Badge>
       <h2 className="sr-only">{intl.formatMessage(messages.headerPlaceholder)}</h2>
       <BookmarkButton
         unitId={unit.id}

diff --git a/src/courseware/course/sequence/messages.js b/src/courseware/course/sequence/messages.js
@@ -31,7 +31,7 @@ const messages = defineMessages({
     defaultMessage: 'There is no content here.',
     description: 'Message shown when there is no content to show a user inside a learning sequence.',
   },
-  optionalContent: {
+  optionalCompletion: {
     id: 'learn.sequence.optionalBlock',
     defaultMessage: 'Optional',
     description: 'Used as a label to indicate that a section, sequence, or unit is optional.',

diff --git a/src/courseware/data/api.js b/src/courseware/data/api.js
@@ -169,7 +169,7 @@ function normalizeSequenceMetadata(sequence) {
       contentType: unit.type,
       graded: unit.graded,
       containsContentTypeGatedContent: unit.contains_content_type_gated_content,
-      optional: unit.optional_content,
+      optional: unit.optional_completion,
     })),
   };
 }