Fixed helper structure

Author: MiroMargineanu

src/components/ProviderInitializer/ProviderInitializer.tsx

@@ -85,7 +85,6 @@ export function ProviderInitializer() {
   const tokenLogin = useSelector(tokenLoginSelector);
   const userAccount = useSelector(accountSelector);
   const nativeAuthConfig = tokenLogin?.nativeAuthConfig;
-  const handleGuardianWarningToast = handleGuardianWarning();
 
   const loginService = useLoginService(
     nativeAuthConfig ? nativeAuthConfig : false
@@ -133,7 +132,7 @@ export function ProviderInitializer() {
 
   useEffect(() => {
     if (isLoggedIn && userAccount.address) {
-      handleGuardianWarningToast(userAccount);
+      handleGuardianWarning(userAccount);
     }
   }, [isLoggedIn, userAccount]);
 

src/components/ProviderInitializer/helpers/handleGuardianWarning.tsx

@@ -9,7 +9,7 @@ import { localStorageKeys } from 'utils/storage/local';
 const DAYS_TO_SHOW_AGAIN_AFTER_DISMISSAL = 3;
 const SECONDS_IN_A_DAY = 24 * 60 * 60;
 
-export const handleGuardianWarning = () => {
+export const handleGuardianWarning = (userAccount: AccountType) => {
   const handleToastDismissal = () => {
     const daysAsSeconds = SECONDS_IN_A_DAY * DAYS_TO_SHOW_AGAIN_AFTER_DISMISSAL;
     const currentTimestamp = Math.floor(Date.now() / 1000);
@@ -21,30 +21,22 @@ export const handleGuardianWarning = () => {
     });
   };
 
-  const insertGuardianWarningToast = () => {
+  const guardianBreachToastDismissTimestamp = storage.local.getItem(
+    localStorageKeys.guardianBreachToastDismissTimestamp
+  );
+
+  const isGuardedAccountPendingChange =
+    userAccount.isGuarded &&
+    userAccount.activeGuardianAddress &&
+    userAccount.pendingGuardianAddress;
+
+  if (isGuardedAccountPendingChange && !guardianBreachToastDismissTimestamp) {
     addNewCustomToast({
       toastId: 'guardianUnconfirmedChangeWarning',
       title: 'Account at risk!',
       component: () => <GuardianWarningToast />,
       icon: faWarning,
       onClose: handleToastDismissal
     });
-  };
-
-  const handleGuardianWarningToast = (userAccount: AccountType) => {
-    const guardianBreachToastDismissTimestamp = storage.local.getItem(
-      localStorageKeys.guardianBreachToastDismissTimestamp
-    );
-
-    const isGuardedAccountPendingChange =
-      userAccount.isGuarded &&
-      userAccount.activeGuardianAddress &&
-      userAccount.pendingGuardianAddress;
-
-    if (isGuardedAccountPendingChange && !guardianBreachToastDismissTimestamp) {
-      insertGuardianWarningToast();
-    }
-  };
-
-  return handleGuardianWarningToast;
+  }
 };