fix(cli): remove hardcoded gas limit from signup and publish

hardcoded gas limit might prevent transaction from succeeding on certain networks fix #1086
privacy-scaling-explorations · Jan 23, 2024 · 0f6139f · 0f6139f
1 parent 24c0bd4
commit 0f6139f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/cli/ts/commands/publish.ts b/cli/ts/commands/publish.ts
@@ -135,9 +135,7 @@ export const publish = async ({
 
   try {
     // submit the message onchain as well as the encryption public key
-    const tx = await pollContract.publishMessage(message.asContractParam(), encKeypair.pubKey.asContractParam(), {
-      gasLimit: 10000000,
-    });
+    const tx = await pollContract.publishMessage(message.asContractParam(), encKeypair.pubKey.asContractParam());
     const receipt = await tx.wait();
 
     if (receipt?.status !== 1) {

diff --git a/cli/ts/commands/signup.ts b/cli/ts/commands/signup.ts
@@ -69,7 +69,7 @@ export const signup = async ({
   let stateIndex = "";
   try {
     // sign up to the MACI contract
-    const tx = await maciContract.signUp(userMaciPubKey.asContractParam(), sgData, ivcpData, { gasLimit: 1000000 });
+    const tx = await maciContract.signUp(userMaciPubKey.asContractParam(), sgData, ivcpData);
     const receipt = await tx.wait();
 
     logYellow(quiet, info(`Transaction hash: ${tx.hash}`));