fix: estimateBatch and sendBatch use same callData

cyberconnecthq · Nov 23, 2023 · 42ee0d9 · 42ee0d9
1 parent 1a0870a
commit 42ee0d9
Showing 1 changed file with 8 additions and 9 deletions.
diff --git a/src/CyberAccount.ts b/src/CyberAccount.ts
@@ -147,14 +147,13 @@ class CyberAccount {
       args: [this.encodeMultiSendCallData(batch)],
     });
 
-    return multiSendCallData;
-    // return this.encodeExecuteCallData(
-    //   {
-    //     to: CyberAccount.MULTI_SEND_ADDRESS,
-    //     data: multiSendCallData,
-    //   },
-    //   1,
-    // );
+    return this.encodeExecuteCallData(
+      {
+        to: CyberAccount.MULTI_SEND_ADDRESS,
+        data: multiSendCallData,
+      },
+      1,
+    );
   }
 
   private encodeMultiSendCallData(batch: UserOperationCallData[]) {
@@ -544,7 +543,7 @@ class CyberAccount {
     const sender = this.address;
     const to = CyberAccount.MULTI_SEND_ADDRESS;
     const value = this.sumBatchTxValues(transactionData).toString();
-    const callData = this.encodeBatchExecuteCallData(transactionData);
+    const callData = this.encodeMultiSendFunctionData(transactionData);
 
     const estimation = await this.paymaster?.estimateUserOperation(
       {