From 7d988b43a68beb930721cefd9b469e26d3452566 Mon Sep 17 00:00:00 2001
From: Navin Karkera <navin@disroot.org>
Date: Mon, 27 Nov 2023 18:18:16 +0530
Subject: [PATCH] refactor: display only changed subsection and units in
 publish modal

---
 src/course-outline/publish-modal/PublishModal.jsx      | 4 ++--
 src/course-outline/publish-modal/PublishModal.test.jsx | 4 ++++
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/course-outline/publish-modal/PublishModal.jsx b/src/course-outline/publish-modal/PublishModal.jsx
index 14a1195f9b..00c3f19f38 100644
--- 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
index e6fad367b2..6dfdbc708a 100644
--- a/src/course-outline/publish-modal/PublishModal.test.jsx
+++ b/src/course-outline/publish-modal/PublishModal.test.jsx
@@ -36,12 +36,14 @@ const currentSectionMock = {
       {
         displayName: 'Subsection 1',
         id: 1,
+        hasChanges: true,
         childInfo: {
           displayName: 'Unit',
           children: [
             {
               id: 11,
               displayName: 'Subsection_1 Unit 1',
+              hasChanges: true,
             },
           ],
         },
@@ -49,12 +51,14 @@ const currentSectionMock = {
       {
         displayName: 'Subsection 2',
         id: 2,
+        hasChanges: true,
         childInfo: {
           displayName: 'Unit',
           children: [
             {
               id: 21,
               displayName: 'Subsection_2 Unit 1',
+              hasChanges: true,
             },
           ],
         },