8000 Gas estimation silenty fails for calls that return `Err` · Issue #6605 · polkadot-js/apps · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Gas estimation silenty fails for calls that return Err #6605

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.

Sign up for GitHub

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

Jump to bottom
Open
athei opened this issue Nov 25, 2021 · 3 comments
Open

Gas estimation silenty fails for calls that return Err #6605

athei opened this issue Nov 25, 2021 · 3 comments

Comments

@athei
Copy link
athei commented Nov 25, 2021

In the UI where the inputs to a call are entered an RPC will be fired off in the background to estimate the gas usage for this particular set of inputs.

Whenever those inputs lead to an error the gas estimation becomes non functional and replaced by a default value. This isn't necessary because the gas_required field is contained in the RPC result no matter the execution status Ok or Err.

What should be done instead:

  • Display the gas estimation no matter the result of the call
  • Display the Result along with the debug buffer to the user so that they know what went wrong
@polkadot-js-bot
Copy link

This issue has been open for 21 days with no activity and is not labelled as an enhancement. It will be closed in 7 days.

@athei
Copy link
Author
athei commented Dec 16, 2021

Please don't close :)

@ap211unitech
Copy link
Member
ap211unitech commented May 13, 2025 •
edited
Loading

Hi @athei , is this still an issue? If yes, can you paste some steps here to reproduce the behaviour. It's little bit unclear to me, what exactly is wrong. Probably, I am looking at wrong place in UI.

If it's already resolved, let me know, I can close it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@jacogr @athei @ap211unitech @polkadot-js-bot
0