Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Address hardcoded gas in UI or calculate it manually #116

Open
judith-Near opened this issue Jun 20, 2022 · 0 comments
Open

Address hardcoded gas in UI or calculate it manually #116

judith-Near opened this issue Jun 20, 2022 · 0 comments

Comments

@judith-Near
Copy link
Contributor

"Given there are a LOT of situations where someone might want to use multisafe TXs to confirm functionCall requests that require 50+ Tgas, I think you should temporarily increase to at least 100Tgas and longer term you can either:

  1. Check the request they are confirming and if the gas field is present you can ADD this amount to the default gas amount.
  2. Provide a UI option to specify the gas.
    But you cannot continue to send users to the wallet with a fixed default gas amount and no option to increase, because you are sending them into a preventable error and retry scenario. This is bad UX."
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant