refactor: display only changed subsection and units in publish modal

open-craft · Dec 5, 2023 · 98fdf30 · 98fdf30
1 parent 0aa1c8b
commit 98fdf30
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/course-outline/publish-modal/PublishModal.jsx b/src/course-outline/publish-modal/PublishModal.jsx
@@ -36,8 +36,8 @@ const PublishModal = ({
       </ModalDialog.Header>
       <ModalDialog.Body>
         <p className="small">{intl.formatMessage(messages.description)}</p>
-        {subSections.map((subSection) => {
-          const units = subSection.childInfo.children;
+        {subSections.filter(subSection => subSection.hasChanges).map((subSection) => {
+          const units = subSection.childInfo.children.filter(unit => unit.hasChanges);
 
           return units.length ? (
             <React.Fragment key={subSection.id}>

diff --git a/src/course-outline/publish-modal/PublishModal.test.jsx b/src/course-outline/publish-modal/PublishModal.test.jsx
@@ -36,25 +36,29 @@ const currentSectionMock = {
       {
         displayName: 'Subsection 1',
         id: 1,
+        hasChanges: true,
         childInfo: {
           displayName: 'Unit',
           children: [
             {
               id: 11,
               displayName: 'Subsection_1 Unit 1',
+              hasChanges: true,
             },
           ],
         },
       },
       {
         displayName: 'Subsection 2',
         id: 2,
+        hasChanges: true,
         childInfo: {
           displayName: 'Unit',
           children: [
             {
               id: 21,
               displayName: 'Subsection_2 Unit 1',
+              hasChanges: true,
             },
           ],
         },