Merge pull request #3032 from devtron-labs/fix/misc-sentry-null

fix: misc sentry null fix

Author: shivani170

src/Pages/App/CreateAppModal/UpdateTemplateConfig.tsx

@@ -69,17 +69,19 @@ const UpdateTemplateConfig = ({
     }
 
     const handleCIConfigParentStateUpdate: CIConfigProps['setParentState'] = (updatedParentStateOrHandler) => {
-        const {
-            ciConfig: { dockerRegistry, dockerRepository },
-        } = updatedParentStateOrHandler
+        if (updatedParentStateOrHandler?.ciConfig) {
+            const {
+                ciConfig: { dockerRegistry, dockerRepository },
+            } = updatedParentStateOrHandler
 
-        handleFormStateChange({
-            action: CreateAppFormStateActionType.updateBuildConfiguration,
-            value: {
-                dockerRegistry,
-                dockerRepository,
-            },
-        })
+            handleFormStateChange({
+                action: CreateAppFormStateActionType.updateBuildConfiguration,
+                value: {
+                    dockerRegistry,
+                    dockerRepository,
+                },
+            })
+        }
     }
 
     const handleGitMaterialsChange: MaterialListProps['handleGitMaterialsChange'] = (updatedGitMaterial, isError) => {

src/components/app/details/triggerView/BuildImageModal/GitInfoMaterial.tsx

@@ -154,7 +154,7 @@ const GitInfoMaterial = ({
         )
         // Not added source not configured check here since ideally this should not be even called at that moment and we are not adding a new material
 
-        if (!newSelectedMaterialItem.result.length) {
+        if (!newSelectedMaterialItem.result?.length) {
             throw new Error('Unable to fetch material details')
         }
 
@@ -209,17 +209,17 @@ const GitInfoMaterial = ({
             updatedMaterialKeys.history = [
                 {
                     commitURL: selectedMaterial.gitURL
-                        ? createGitCommitUrl(selectedMaterial.gitURL, commitHistoryResult.Commit)
+                        ? createGitCommitUrl(selectedMaterial.gitURL, commitHistoryResult?.Commit)
                         : '',
-                    commit: commitHistoryResult.Commit || '',
-                    author: commitHistoryResult.Author || '',
-                    date: commitHistoryResult.Date ? handleUTCTime(commitHistoryResult.Date, false) : '',
-                    message: commitHistoryResult.Message || '',
-                    changes: commitHistoryResult.Changes || [],
+                    commit: commitHistoryResult?.Commit || '',
+                    author: commitHistoryResult?.Author || '',
+                    date: commitHistoryResult?.Date ? handleUTCTime(commitHistoryResult.Date, false) : '',
+                    message: commitHistoryResult?.Message || '',
+                    changes: commitHistoryResult?.Changes || [],
                     showChanges: true,
-                    webhookData: commitHistoryResult.WebhookData,
-                    isSelected: !commitHistoryResult.Excluded,
-                    excluded: commitHistoryResult.Excluded,
+                    webhookData: commitHistoryResult?.WebhookData,
+                    isSelected: !commitHistoryResult?.Excluded,
+                    excluded: commitHistoryResult?.Excluded,
                 },
             ]