feat: Enhance BulkDeployModal and MaterialListEmptyState for improved material handling and error management

Author: AbhishekA1509

src/components/app/details/triggerView/DeployImageModal/BulkDeployModal.tsx

@@ -236,6 +236,7 @@ const BulkDeployModal = ({ handleClose, stageType, workflows, isVirtualEnvironme
             ...prev,
             [selectedAppId]: {
                 ...prev[selectedAppId],
+                materialResponse: response[selectedAppId]?.materialResponse,
                 warningMessage,
                 materialError,
                 deploymentWindowMetadata,
@@ -251,6 +252,10 @@ const BulkDeployModal = ({ handleClose, stageType, workflows, isVirtualEnvironme
         }))
     }
 
+    const reloadMaterials = async () => {
+        await reloadOrSearchSelectedApp()
+    }
+
     const handleLoadOlderImages = async () => {
         try {
             // Even if user changes selectedAppId this will persist since a state closure
@@ -452,29 +457,31 @@ const BulkDeployModal = ({ handleClose, stageType, workflows, isVirtualEnvironme
     const handleTagChange: DeployImageContentProps['handleTagChange'] = (tagOption) => {
         setSelectedImageTagOption(tagOption)
 
-        const selectedApp = appInfoMap[selectedAppId]
-        const updatedMaterials = getUpdatedMaterialsForTagSelection(
-            tagOption.value,
-            selectedApp.materialResponse?.materials || [],
-        )
-        const { tagsWarning, updatedMaterials: newMaterials } = updatedMaterials
+        setAppInfoMap((prev) => {
+            const updatedAppInfoMap = structuredClone(prev)
+            Object.values(updatedAppInfoMap).forEach((appDetails) => {
+                const { tagsWarning, updatedMaterials } = getUpdatedMaterialsForTagSelection(
+                    tagOption.value,
+                    appDetails.materialResponse?.materials || [],
+                )
 
-        const { tagsWarning: previousTagWarning } = getUpdatedMaterialsForTagSelection(
-            selectedImageTagOption.value,
-            selectedApp.materialResponse?.materials || [],
-        )
+                const { tagsWarning: previousTagWarning } = getUpdatedMaterialsForTagSelection(
+                    selectedImageTagOption.value,
+                    appDetails.materialResponse?.materials || [],
+                )
 
-        setAppInfoMap((prev) => ({
-            ...prev,
-            [selectedAppId]: {
-                ...prev[selectedAppId],
-                materialResponse: {
-                    ...prev[selectedAppId].materialResponse,
-                    materials: newMaterials,
-                },
-                warningMessage: previousTagWarning ? tagsWarning : prev[selectedAppId].warningMessage,
-            },
-        }))
+                updatedAppInfoMap[appDetails.appId] = {
+                    ...appDetails,
+                    materialResponse: {
+                        ...appDetails.materialResponse,
+                        materials: updatedMaterials,
+                    },
+                    warningMessage:
+                        previousTagWarning || !appDetails.warningMessage ? tagsWarning : appDetails.warningMessage,
+                }
+            })
+            return updatedAppInfoMap
+        })
     }
 
     const changeApp: DeployImageContentProps['changeApp'] = (appId) => {
@@ -601,7 +608,7 @@ const BulkDeployModal = ({ handleClose, stageType, workflows, isVirtualEnvironme
                 appName={appName}
                 isSecurityModuleInstalled={isSecurityModuleInstalled}
                 envName={envName}
-                reloadMaterials={reloadOrSearchSelectedApp}
+                reloadMaterials={reloadMaterials}
                 parentEnvironmentName={parentEnvironmentName}
                 isVirtualEnvironment={isVirtualEnvironment}
                 loadOlderImages={handleLoadOlderImages}

src/components/app/details/triggerView/DeployImageModal/MaterialListEmptyState.tsx

@@ -42,7 +42,7 @@ const MaterialListEmptyState = ({
     const isApprovalConfigured = getIsApprovalPolicyConfigured(
         materialResponse?.deploymentApprovalInfo?.approvalConfigData,
     )
-    const areNoMoreImagesPresent = materialResponse && materialResponse.materials.length >= materialResponse.totalCount
+    const areNoMoreImagesPresent = materialResponse && material.length >= materialResponse.totalCount
     const resourceFilters = materialResponse?.resourceFilters ?? []
 
     const clearSearch = () => {

src/components/app/details/triggerView/DeployImageModal/utils.tsx

@@ -520,7 +520,7 @@ export const getBulkCDDetailsMapFromResponse: GetBulkCDDetailsMapFromResponseTyp
     cdMaterialResponseList.forEach((materialResponse, index) => {
         const { appId } = validWorkflows[index]
 
-        if (materialResponse.status === PromiseAllStatusType.FULFILLED) {
+        if (materialResponse.status === PromiseAllStatusType.FULFILLED && materialResponse.value) {
             const { tagsWarning, updatedMaterials } = getUpdatedMaterialsForTagSelection(
                 selectedTagName,
                 materialResponse.value.materials,
@@ -552,7 +552,8 @@ export const getBulkCDDetailsMapFromResponse: GetBulkCDDetailsMapFromResponseTyp
                 materialResponse: {} as CDMaterialResponseType,
                 deploymentWindowMetadata: {} as DeploymentWindowProfileMetaData,
                 deployViewState: structuredClone(INITIAL_DEPLOY_VIEW_STATE),
-                materialError: materialResponse.reason,
+                materialError:
+                    materialResponse.status === PromiseAllStatusType.REJECTED ? materialResponse.reason : null,
                 areMaterialsLoading: false,
             }
         }