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

Get gas price suggestion from the correct block #817

Merged
merged 6 commits into from
Jan 31, 2020

Conversation

mrsmkl
Copy link
Contributor

@mrsmkl mrsmkl commented Jan 21, 2020

Description

Get the gas price for the block specified in the RPC call so that it doesn't fail transaction pre-check.

Tested

Only manual testing for now. Perhaps an e2e test could be added.

Other changes

Related issues

Backwards compatibility

@mrsmkl mrsmkl requested a review from asaj as a code owner January 21, 2020 08:13
Copy link
Contributor

@lucasege lucasege left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding err check, lgtm

@lucasege lucasege assigned mrsmkl and unassigned lucasege Jan 29, 2020
@asaj asaj removed their assignment Jan 30, 2020
@celo-ci-bot-user celo-ci-bot-user merged commit afe2f7b into celo-org:master Jan 31, 2020
mcortesi pushed a commit that referenced this pull request Jan 31, 2020
# Conflicts:
#	internal/ethapi/api.go
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

Successfully merging this pull request may close these issues.

In RPC, if no gas price is specified, make sure that suitable gas price is selected
4 participants