Merge pull request #3 from StauroXYZ/fix/fix_call_for_delegate

fix: remove the delegatecall because it is not needed for the delegate
StauroDEV · Nov 9, 2023 · 6de27b8 · 6de27b8
2 parents a10d8ea + 353aaf1
commit 6de27b8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/src/actions/ens.ts b/src/actions/ens.ts
@@ -25,8 +25,8 @@ export const ensAction = async (
   cid: string,
   domain: string,
   {
-    chain: chainName, safe: safeAddress, operationType, rpcUrl
-  }: { chain: ChainName; } & Partial<{ safe: Address | EIP3770Address, operationType: OperationType; rpcUrl:string }>,
+    chain: chainName, safe: safeAddress, rpcUrl
+  }: { chain: ChainName; } & Partial<{ safe: Address | EIP3770Address, rpcUrl:string }>,
 ) => {
   const chain = chainName === 'mainnet' ? mainnet : goerli
   const publicClient = createPublicClient({
@@ -89,7 +89,7 @@ export const ensAction = async (
     const txData = {
       ...request,
       to: request.to as Address,
-      operation: operationType ?? OperationType.Call,
+      operation: OperationType.Call,
       gasPrice: request.gasPrice ?? 0n,
       nonce
     }

diff --git a/src/cli.ts b/src/cli.ts
@@ -7,7 +7,6 @@ import { statusAction } from './actions/status.js'
 import { deployAction } from './actions/deploy.js'
 
 import './polyfills/fetch.js'
-import { OperationType } from '@stauro/piggybank/types'
 
 const cli = cac('blumen')
 
@@ -34,7 +33,6 @@ cli
   )
   .option('--chain <chain>', 'Chain to use', { default: 'mainnet' })
   .option('--safe <safe>', 'Deploy using a Safe multisig wallet')
-  .option('--operation-type <number>', 'Operation type to use for a Safe multisig wallet (0 - Call, 1 - DelegateCall)', { default: OperationType.Call })
   .option('--rpc-url <url>', 'Custom Ethereum RPC')
   .action(ensAction)