diff --git a/packages/features/accounts/src/components/personal-account-settings/mfa/multi-factor-auth-list.tsx b/packages/features/accounts/src/components/personal-account-settings/mfa/multi-factor-auth-list.tsx index cf74b4d61..87ea95a4f 100644 --- a/packages/features/accounts/src/components/personal-account-settings/mfa/multi-factor-auth-list.tsx +++ b/packages/features/accounts/src/components/personal-account-settings/mfa/multi-factor-auth-list.tsx @@ -46,17 +46,25 @@ import { Trans } from '@kit/ui/trans'; import { MultiFactorAuthSetupDialog } from './multi-factor-auth-setup-dialog'; -const MAX_FACTOR_COUNT = 10; - export function MultiFactorAuthFactorsList(props: { userId: string }) { + return ( +
+ + +
+ +
+
+ ); +} + +function FactorsTableContainer(props: { userId: string }) { const { data: factors, isLoading, isError, } = useFetchAuthFactors(props.userId); - const [unEnrolling, setUnenrolling] = useState(); - if (isLoading) { return (
@@ -103,37 +111,11 @@ export function MultiFactorAuthFactorsList(props: { userId: string }) { - -
- -
); } - const canAddNewFactors = allFactors.length < MAX_FACTOR_COUNT; - - return ( -
- - - -
- -
-
- - - {(factorId) => ( - setUnenrolling(undefined)} - /> - )} - -
- ); + return ; } function ConfirmUnenrollFactorModal( @@ -157,7 +139,7 @@ function ConfirmUnenrollFactorModal( const errorCode = response.data; throw t(`auth:errors.${errorCode}`, { - defaultValue: t(`account:unenrollFactorError`) + defaultValue: t(`account:unenrollFactorError`), }); } }); @@ -205,75 +187,89 @@ function ConfirmUnenrollFactorModal( } function FactorsTable({ - setUnenrolling, factors, + userId, }: React.PropsWithChildren<{ - setUnenrolling: (factorId: string) => void; factors: Factor[]; + userId: string; }>) { + const [unEnrolling, setUnenrolling] = useState(); + return ( - - - - - - - - - - - - + <> +
+ + + + + + + + + + + - - - - - - {factors.map((factor) => ( - - - {factor.friendly_name} - - - - - {factor.factor_type} - - - - - - + - ))} - -
- - {factor.status} - - - - - - - - - - - - - -
+ + + + {factors.map((factor) => ( + + + {factor.friendly_name} + + + + + {factor.factor_type} + + + + + + {factor.status} + + + + + + + + + + + + + + + + + + ))} + + + + + {(factorId) => ( + setUnenrolling(undefined)} + /> + )} + + ); } @@ -291,13 +287,13 @@ function useUnenrollFactor(userId: string) { return { success: false as const, data: error.code as string, - } + }; } return { success: true as const, data, - } + }; }; return useMutation({ diff --git a/packages/features/accounts/src/components/personal-account-settings/mfa/multi-factor-auth-setup-dialog.tsx b/packages/features/accounts/src/components/personal-account-settings/mfa/multi-factor-auth-setup-dialog.tsx index b3c02a559..728186ac4 100644 --- a/packages/features/accounts/src/components/personal-account-settings/mfa/multi-factor-auth-setup-dialog.tsx +++ b/packages/features/accounts/src/components/personal-account-settings/mfa/multi-factor-auth-setup-dialog.tsx @@ -53,7 +53,7 @@ export function MultiFactorAuthSetupDialog(props: { userId: string }) { const onEnrollSuccess = useCallback(() => { setIsOpen(false); - return toast.success(t(`multiFactorSetupSuccess`)); + return toast.success(t(`account:multiFactorSetupSuccess`)); }, [t]); return (