fix: correct optional chaining for gitProvider in MaterialForm and update material type in UpdateMaterialPayload

Author: AbhishekA1509

src/components/material/MaterialForm.tsx

@@ -52,7 +52,7 @@ const MaterialForm = ({
         if (!url.length) {
             return 'This is a required field'
         }
-        if (isAWSCodeCommitURL(formMaterial?.gitProvider?.url)) {
+        if (isAWSCodeCommitURL(formMaterial.gitProvider?.url)) {
             if (isAWSCodeCommitURL(url)) {
                 return undefined
             }
@@ -178,7 +178,7 @@ const MaterialForm = ({
             material
                 ? {
                       gitProvider: isGitProviderValid(material.gitProvider),
-                      url: isGitUrlValid(material.url, formMaterial?.gitProvider?.id),
+                      url: isGitUrlValid(material.url, formMaterial.gitProvider?.id),
                       checkoutPath: isCheckoutPathValid(material.checkoutPath),
                   }
                 : {

src/components/material/MaterialList.tsx

@@ -112,14 +112,15 @@ const MaterialList = ({
         return undefined
     }
 
-    const handleSingleGitMaterialUpdate = (id: GitMaterialType['id']) => (updatedMaterial, isError) => {
-        if (handleGitMaterialsChange) {
-            handleGitMaterialsChange(
-                materials.map((mat) => (mat.id === id ? updatedMaterial : mat)),
-                isError,
-            )
+    const handleSingleGitMaterialUpdate =
+        (id: GitMaterialType['id']) => (updatedMaterial: GitMaterialType, isError: boolean) => {
+            if (handleGitMaterialsChange) {
+                handleGitMaterialsChange(
+                    materials.map((mat) => (mat.id === id ? updatedMaterial : mat)),
+                    isError,
+                )
+            }
         }
-    }
 
     const renderPageHeader = () => (
         <>

src/components/material/material.types.ts

@@ -149,5 +149,5 @@ export interface CreateMaterialPayload {
 
 export interface UpdateMaterialPayload {
     appId: number
-    material: Omit<UpsertMaterialItemPayload, 'id'> & { id: number }
+    material: UpsertMaterialItemPayload & { id: number }
 }