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

[ctrlc03]

Description

hardcoded gas limit might prevent transaction from succeeding on certain networks

Related issue(s)

fix #1086

Confirmation

• I have read and understand MACI's contributor guidelines and code of conduct.
• I have read and understand MACI's GitHub processes.
• I have read and understand MACI's testing guide.

[netlify]

✅ Deploy Preview for maci-typedoc ready!

Name	Link
🔨 Latest commit	1bf3a4b
🔍 Latest deploy log	https://app.netlify.com/sites/maci-typedoc/deploys/65af9f67286ae8000754932c
😎 Deploy Preview	https://deploy-preview-1087--maci-typedoc.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[kittybest]

thanks!
I'm curious that does that mean the gas spent more than we set up?

[ctrlc03]

thanks! I'm curious that does that mean the gas spent more than we set up?

yes, usually you would set a gas limit when you want the tx to be sent regardless of whether it would fail or not. If you don't specify, and if the tx fails it will just not send it and give you the error. In our case, it seems that on this test network signin up spends more than 1M gas, so the tx would go through but fail because of not enough gas (limit was set to 1M)

[0xmad]

@ctrlc03 I created a task for this case #1053. So the idea is to use static calls to check if transaction can be failed without paying for the gas.