From 3c590d70c3945f667e170e3a0d376ef396349fe6 Mon Sep 17 00:00:00 2001
From: Sebastian Gerske <13647606+H34D@users.noreply.github.com>
Date: Thu, 2 Mar 2023 16:34:37 +0100
Subject: [PATCH] refactor

---
 src/provider/masa-context-provider.tsx | 16 ++--------------
 src/provider/modules/modal/modal.ts    | 16 ++++++++++++++++
 src/provider/use-metamask.ts           |  3 ++-
 3 files changed, 20 insertions(+), 15 deletions(-)

diff --git a/src/provider/masa-context-provider.tsx b/src/provider/masa-context-provider.tsx
index a516d338..d3883aef 100644
--- a/src/provider/masa-context-provider.tsx
+++ b/src/provider/masa-context-provider.tsx
@@ -115,6 +115,7 @@ export const MasaContextProvider = ({
     closeModal,
     forcedPage,
     setForcedPage,
+    openMintSoulnameModal,
   } = useModal(
     masaInstance,
     isLoggedIn,
@@ -157,20 +158,6 @@ export const MasaContextProvider = ({
     [setModalOpen, setModalCallback, setScope, setForcedPage, forcedPage]
   );
 
-  const openMintSoulnameModal = useCallback(
-    (mintCallback?: () => void) => {
-      setForcedPage?.('createSoulname');
-      setModalOpen(true);
-      const cb = () => {
-        setForcedPage?.(null);
-        if (mintCallback) mintCallback();
-      };
-
-      setModalCallback(() => cb);
-    },
-    [setForcedPage, setModalOpen, setModalCallback]
-  );
-
   useEffect(() => {
     const loadMasa = async (): Promise<void> => {
       if (!provider) return;
@@ -221,6 +208,7 @@ export const MasaContextProvider = ({
     forcedPage,
     setForcedPage,
     openMintSoulnameModal,
+
     // wallet
     walletAddress,
     isWalletLoading,
diff --git a/src/provider/modules/modal/modal.ts b/src/provider/modules/modal/modal.ts
index a543150b..74d95c01 100644
--- a/src/provider/modules/modal/modal.ts
+++ b/src/provider/modules/modal/modal.ts
@@ -15,6 +15,7 @@ export const useModal = (
   setModalCallback: (callback: () => void) => void;
   forcedPage: string | null;
   setForcedPage?: (page: null | string) => void;
+  openMintSoulnameModal: (mintCallback?: () => void) => void;
 } => {
   const [isModalOpen, setModalOpen] = useState(false);
   const [modalCallback, setModalCallback] = useState<(() => void) | null>(null);
@@ -51,6 +52,20 @@ export const useModal = (
     ]
   );
 
+  const openMintSoulnameModal = useCallback(
+    (mintCallback?: () => void) => {
+      setForcedPage?.('createSoulname');
+      setModalOpen(true);
+      const cb = () => {
+        setForcedPage?.(null);
+        if (mintCallback) mintCallback();
+      };
+
+      setModalCallback(() => cb);
+    },
+    [setForcedPage, setModalOpen, setModalCallback]
+  );
+
   return {
     isModalOpen,
     closeModal,
@@ -58,5 +73,6 @@ export const useModal = (
     setModalCallback,
     forcedPage,
     setForcedPage,
+    openMintSoulnameModal,
   };
 };
diff --git a/src/provider/use-metamask.ts b/src/provider/use-metamask.ts
index 5a636ed9..4266c7ac 100644
--- a/src/provider/use-metamask.ts
+++ b/src/provider/use-metamask.ts
@@ -37,8 +37,9 @@ export const useMetamask = ({
           console.error('No accounts returned from metamask');
         }
       } catch (error) {
-        if (error instanceof Error)
+        if (error instanceof Error) {
           console.error('Failed to connect to metamask!', error.message);
+        }
       }
 
       const signer = getWeb3Provider()?.getSigner();