refactor: simplify authentication form submission logic across multiple identity sections

frontend/src/pages/organization/AccessManagementPage/components/OrgIdentityTab/components/IdentitySection/IdentityAliCloudAuthForm.tsx

@@ -137,7 +137,6 @@ export const IdentityAliCloudAuthForm = ({
     accessTokenNumUsesLimit,
     accessTokenTrustedIps
   }: FormData) => {
-    try {
     if (!identityId) return;
 
     if (data) {
@@ -170,12 +169,6 @@ export const IdentityAliCloudAuthForm = ({
     });
 
     reset();
-    } catch {
-      createNotification({
-        text: `Failed to ${isUpdate ? "update" : "configure"} identity`,
-        type: "error"
-      });
-    }
   };
 
   return (

frontend/src/pages/organization/AccessManagementPage/components/OrgIdentityTab/components/IdentitySection/IdentityAuthTemplateModal.tsx

@@ -103,7 +103,6 @@ export const IdentityAuthTemplateModal = ({ popUp, handlePopUpToggle }: Props) =
   const selectedMethod = watch("method");
 
   const onFormSubmit = async (data: FormData) => {
-    try {
     if (isEdit && template) {
       await updateTemplate({
         templateId: template.id,
@@ -142,17 +141,6 @@ export const IdentityAuthTemplateModal = ({ popUp, handlePopUpToggle }: Props) =
 
     handlePopUpToggle(isEdit ? "editTemplate" : "createTemplate", false);
     reset();
-    } catch (err) {
-      console.error(err);
-      const error = err as any;
-      const text =
-        error?.response?.data?.message ?? `Failed to ${isEdit ? "update" : "create"} auth template`;
-
-      createNotification({
-        text,
-        type: "error"
-      });
-    }
   };
 
   const handleClose = () => {

frontend/src/pages/organization/AccessManagementPage/components/OrgIdentityTab/components/IdentitySection/IdentityAwsAuthForm.tsx

@@ -147,7 +147,6 @@ export const IdentityAwsAuthForm = ({
     accessTokenNumUsesLimit,
     accessTokenTrustedIps
   }: FormData) => {
-    try {
     if (!identityId) return;
 
     if (data) {
@@ -184,12 +183,6 @@ export const IdentityAwsAuthForm = ({
     });
 
     reset();
-    } catch {
-      createNotification({
-        text: `Failed to ${isUpdate ? "update" : "configure"} identity`,
-        type: "error"
-      });
-    }
   };
 
   return (

frontend/src/pages/organization/AccessManagementPage/components/OrgIdentityTab/components/IdentitySection/IdentityAzureAuthForm.tsx

@@ -143,7 +143,6 @@ export const IdentityAzureAuthForm = ({
     accessTokenNumUsesLimit,
     accessTokenTrustedIps
   }: FormData) => {
-    try {
     if (!identityId) return;
 
     if (data) {
@@ -180,12 +179,6 @@ export const IdentityAzureAuthForm = ({
     });
 
     reset();
-    } catch {
-      createNotification({
-        text: `Failed to ${isUpdate ? "update" : "configure"} identity`,
-        type: "error"
-      });
-    }
   };
 
   return (

frontend/src/pages/organization/AccessManagementPage/components/OrgIdentityTab/components/IdentitySection/IdentityGcpAuthForm.tsx

@@ -152,7 +152,6 @@ export const IdentityGcpAuthForm = ({
     accessTokenNumUsesLimit,
     accessTokenTrustedIps
   }: FormData) => {
-    try {
     if (!identityId) return;
 
     if (data) {
@@ -191,12 +190,6 @@ export const IdentityGcpAuthForm = ({
     });
 
     reset();
-    } catch {
-      createNotification({
-        text: `Failed to ${isUpdate ? "update" : "configure"} identity`,
-        type: "error"
-      });
-    }
   };
 
   return (

frontend/src/pages/organization/AccessManagementPage/components/OrgIdentityTab/components/IdentitySection/IdentityJwtAuthForm.tsx

@@ -217,7 +217,6 @@ export const IdentityJwtAuthForm = ({
     boundClaims,
     boundSubject
   }: FormData) => {
-    try {
     if (!identityId) {
       return;
     }
@@ -266,12 +265,6 @@ export const IdentityJwtAuthForm = ({
     });
 
     reset();
-    } catch {
-      createNotification({
-        text: `Failed to ${isUpdate ? "update" : "configure"} identity`,
-        type: "error"
-      });
-    }
   };
 
   return (

frontend/src/pages/organization/AccessManagementPage/components/OrgIdentityTab/components/IdentitySection/IdentityKubernetesAuthForm.tsx

@@ -311,7 +311,6 @@ export const IdentityKubernetesAuthForm = ({
     tokenReviewMode,
     accessTokenTrustedIps
   }: FormData) => {
-    try {
     if (!identityId) return;
 
     if (data) {
@@ -370,12 +369,6 @@ export const IdentityKubernetesAuthForm = ({
     });
 
     reset();
-    } catch {
-      createNotification({
-        text: `Failed to ${isUpdate ? "update" : "configure"} identity`,
-        type: "error"
-      });
-    }
   };
 
   const tokenReviewMode = watch("tokenReviewMode");

frontend/src/pages/organization/AccessManagementPage/components/OrgIdentityTab/components/IdentitySection/IdentityLdapAuthForm.tsx

@@ -316,7 +316,6 @@ export const IdentityLdapAuthForm = ({
   }, [subscription, handlePopUpOpen, handlePopUpToggle]);
 
   const onFormSubmit = async (formData: FormData) => {
-    try {
     if (!identityId) return;
 
     const {
@@ -387,12 +386,6 @@ export const IdentityLdapAuthForm = ({
     });
 
     reset();
-    } catch {
-      createNotification({
-        text: `Failed to ${isUpdate ? "update" : "configure"} identity`,
-        type: "error"
-      });
-    }
   };
 
   return (

frontend/src/pages/organization/AccessManagementPage/components/OrgIdentityTab/components/IdentitySection/IdentityLinkForm.tsx

@@ -42,7 +42,6 @@ export const IdentityLinkForm = ({ onClose }: Props) => {
   });
 
   const onFormSubmit = async ({ identity, role }: FormData) => {
-    try {
     await createMutateAsync({
       identityId: identity.id,
       roles: [{ role: role.slug, isTemporary: false }]
@@ -57,16 +56,6 @@ export const IdentityLinkForm = ({ onClose }: Props) => {
         identityId: identity.id
       }
     });
-    } catch (err) {
-      console.error(err);
-      const error = err as any;
-      const text = error?.response?.data?.message ?? "Failed to link identity";
-
-      createNotification({
-        text,
-        type: "error"
-      });
-    }
   };
 
   return (

frontend/src/pages/organization/AccessManagementPage/components/OrgIdentityTab/components/IdentitySection/IdentityModal.tsx

@@ -108,7 +108,6 @@ export const IdentityModal = ({ popUp, handlePopUpToggle }: Props) => {
   }, [popUp?.identity?.data, roles]);
 
   const onFormSubmit = async ({ name, role, metadata, hasDeleteProtection }: FormData) => {
-    try {
     const identity = popUp?.identity?.data as {
       identityId: string;
       name: string;
@@ -171,18 +170,6 @@ export const IdentityModal = ({ popUp, handlePopUpToggle }: Props) => {
     });
 
     reset();
-    } catch (err) {
-      console.error(err);
-      const error = err as any;
-      const text =
-        error?.response?.data?.message ??
-        `Failed to ${popUp?.identity?.data ? "update" : "create"} identity`;
-
-      createNotification({
-        text,
-        type: "error"
-      });
-    }
   };
 
   return (

frontend/src/pages/organization/AccessManagementPage/components/OrgIdentityTab/components/IdentitySection/IdentityOciAuthForm.tsx

@@ -149,7 +149,6 @@ export const IdentityOciAuthForm = ({
     accessTokenNumUsesLimit,
     accessTokenTrustedIps
   }: FormData) => {
-    try {
     if (!identityId) return;
 
     if (data) {
@@ -184,12 +183,6 @@ export const IdentityOciAuthForm = ({
     });
 
     reset();
-    } catch {
-      createNotification({
-        text: `Failed to ${isUpdate ? "update" : "configure"} identity`,
-        type: "error"
-      });
-    }
   };
 
   return (

frontend/src/pages/organization/AccessManagementPage/components/OrgIdentityTab/components/IdentitySection/IdentityOidcAuthForm.tsx

@@ -201,7 +201,6 @@ export const IdentityOidcAuthForm = ({
     claimMetadataMapping,
     boundSubject
   }: FormData) => {
-    try {
     if (!identityId) {
       return;
     }
@@ -252,12 +251,6 @@ export const IdentityOidcAuthForm = ({
     });
 
     reset();
-    } catch {
-      createNotification({
-        text: `Failed to ${isUpdate ? "update" : "configure"} identity`,
-        type: "error"
-      });
-    }
   };
 
   return (