Comment out broken step, re-enable branch deploy for demo

getwax · May 29, 2024 · 02adb41 · 02adb41
1 parent 730095b
commit 02adb41
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 22 deletions.
diff --git a/.github/workflows/email-recovery-demo.yml b/.github/workflows/email-recovery-demo.yml
@@ -3,7 +3,7 @@ name: packages/demos/email-recovery
 
 on:
   push:
-    branches: ['main']
+    branches: ['main', 'updated-ui']
     paths:
         - packages/demos/email-recovery/**
 
@@ -49,7 +49,8 @@ jobs:
       - name: Build
         env:
           VITE_WALLET_CONNECT_PROJECT_ID: ${{ secrets.VITE_WALLET_CONNECT_PROJECT_ID }}
-        run: VITE_WALLET_CONNECT_PROJECT_ID=${VITE_WALLET_CONNECT_PROJECT_ID} yarn build
+          VITE_OWNER_1_PRIVATE_KEY: ${{ secrets.VITE_OWNER_1_PRIVATE_KEY }}
+        run: VITE_WALLET_CONNECT_PROJECT_ID=${VITE_WALLET_CONNECT_PROJECT_ID} VITE_OWNER_1_PRIVATE_KEY=${VITE_OWNER_1_PRIVATE_KEY} yarn build
 
       - name: Setup Pages
         uses: actions/configure-pages@v3

diff --git a/packages/demos/email-recovery/src/components/EnableSafeModule.tsx b/packages/demos/email-recovery/src/components/EnableSafeModule.tsx
@@ -101,26 +101,26 @@ const EnableSafeModule = () => {
 
     // TODO This step currently reverts as it needs to be run via ERC-4337 Entrypoint (UserOp)
     // Since this is likely the only UserOp we need to run, can directly submit to entrypoint
-    console.debug("4", "Install email recovery module");
-
-    // TODO Move to env
-    // const bundlerUrl = "https://public.stackup.sh/api/v1/node/base-sepolia";
-
-    // TODO Move to ConfigureSafeModule component?
-    // TODO Check this is not already installed, can just directly submit to entrypoint
-    const executorModuleTypeId = 2; // Executor
-    await writeContractAsync({
-      abi: safe7579Abi,
-      address: safe7579 as `0x{string}`,
-      functionName: "installModule",
-      args: [
-        executorModuleTypeId,
-        safeRecoveryModule,
-        "", // TODO Check this install data is no longer needed
-      ]
-    });
-
-    console.debug("Done!");
+    // console.debug("4", "Install email recovery module");
+
+    // // TODO Move to env
+    // // const bundlerUrl = "https://public.stackup.sh/api/v1/node/base-sepolia";
+
+    // // TODO Move to ConfigureSafeModule component?
+    // // TODO Check this is not already installed, can just directly submit to entrypoint
+    // const executorModuleTypeId = 2; // Executor
+    // await writeContractAsync({
+    //   abi: safe7579Abi,
+    //   address: safe7579 as `0x{string}`,
+    //   functionName: "installModule",
+    //   args: [
+    //     executorModuleTypeId,
+    //     safeRecoveryModule,
+    //     "", // TODO Check this install data is no longer needed
+    //   ]
+    // });
+
+    // console.debug("Done!");
 
   }, [address, writeContractAsync]);