[Human App] fix: error notification is displayed twice when should be…

… once (#3110)
humanprotocol · Feb 21, 2025 · 060c601 · 060c601
1 parent f9a2c4d
commit 060c601
Showing 1 changed file with 24 additions and 13 deletions.
diff --git a/packages/apps/human-app/frontend/src/modules/worker/components/profile/profile.page.tsx b/packages/apps/human-app/frontend/src/modules/worker/components/profile/profile.page.tsx
@@ -14,23 +14,34 @@ import { useAuthenticatedUser } from '@/modules/auth/hooks/use-authenticated-use
 export function WorkerProfilePage() {
   const { user } = useAuthenticatedUser();
   const isMobile = useIsMobile();
-  const { isConnected } = useWalletConnect();
+  const { isConnected, initializing, web3ProviderMutation } =
+    useWalletConnect();
   const { showNotification } = useNotification();
 
-  const setNotifications = () => {
-    if (user.wallet_address) {
-      return;
+  useEffect(() => {
+    if (initializing) return;
+
+    if (!isConnected || !user.wallet_address) {
+      showNotification({
+        type: TopNotificationType.WARNING,
+        message: t('worker.profile.topNotifications.completeSteps'),
+      });
     }
-    showNotification({
-      type: TopNotificationType.WARNING,
-      message: t('worker.profile.topNotifications.completeSteps'),
-    });
-  };
 
-  useEffect(() => {
-    setNotifications();
-    // eslint-disable-next-line react-hooks/exhaustive-deps -- call this once
-  }, [isConnected]);
+    if (web3ProviderMutation.isError && web3ProviderMutation.failureReason) {
+      showNotification({
+        type: TopNotificationType.WARNING,
+        message: web3ProviderMutation.failureReason.message,
+      });
+    }
+  }, [
+    isConnected,
+    initializing,
+    web3ProviderMutation.failureReason,
+    web3ProviderMutation.isError,
+    user.wallet_address,
+    showNotification,
+  ]);
 
   return (
     <Paper