From 6d3359fff427c2e82bd59c4cfe546c3189eef3ab Mon Sep 17 00:00:00 2001
From: Daniel Valenzuela <daniel.valenzuela@opencraft.com>
Date: Thu, 29 Feb 2024 03:27:46 -0300
Subject: [PATCH] refactor: rename to optional completion

---
 src/course-home/data/api.js                  | 4 ++--
 src/course-home/outline-tab/Section.jsx      | 2 +-
 src/course-home/outline-tab/SequenceLink.jsx | 2 +-
 src/course-home/outline-tab/messages.js      | 2 +-
 src/courseware/course/sequence/Unit.jsx      | 2 +-
 src/courseware/course/sequence/messages.js   | 2 +-
 src/courseware/data/api.js                   | 2 +-
 7 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/course-home/data/api.js b/src/course-home/data/api.js
index 76fd785d32..633b11c20c 100644
--- 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
index d93265122d..80a45d63c5 100644
--- 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
index 3cfc8356b6..59c28299f2 100644
--- 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
index 98a76ffe5d..d3b0ac5e5c 100644
--- 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
index 1be69e0621..415f4037a3 100644
--- 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
index a35665249e..e050d5222a 100644
--- 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
index cacd62b1ab..f0bdf83595 100644
--- 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,
     })),
   };
 }