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

Ensures that transactions cannot be confirmed if gas limit is below 21000 #6625

Merged
merged 1 commit into from
May 23, 2019

Conversation

danjm
Copy link
Contributor

@danjm danjm commented May 17, 2019

Fixes #6373.

Ensures that transactions cannot be confirmed if gas limit is below 21000.

@danjm danjm requested a review from whymarrh as a code owner May 17, 2019 07:23
@metamaskbot
Copy link
Collaborator

Builds ready [3a0b768]: chrome, firefox, edge, opera

@@ -30,6 +30,7 @@ export default class AdvancedTabContent extends Component {
super(props)

this.debouncedGasLimitReset = debounce((dVal) => {
console.log('------------123123123123123123----------------')
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: console.log

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@whymarrh this is fixed

@danjm danjm force-pushed the ensure-sufficient-gas-limit branch from 3a0b768 to b982e71 Compare May 22, 2019 14:26
@metamaskbot
Copy link
Collaborator

Builds ready [b982e71]: chrome, firefox, edge, opera

@danjm danjm force-pushed the ensure-sufficient-gas-limit branch from b982e71 to 6e751b6 Compare May 23, 2019 03:18
@metamaskbot
Copy link
Collaborator

Builds ready [6e751b6]: chrome, firefox, edge, opera

@danjm danjm merged commit 5e2fd8a into develop May 23, 2019
@whymarrh whymarrh deleted the ensure-sufficient-gas-limit branch May 23, 2019 16:46
@danfinlay danfinlay mentioned this pull request Jun 3, 2019
@backmen
Copy link

backmen commented Feb 2, 2023

We cannot send coins from desktop extension on our chain, because we have lower gas fee than Ethereum. That should be fixed..
#16593

@github-actions
Copy link
Contributor

github-actions bot commented Feb 2, 2023

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

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.

UI - Send Transaction does not validate minimum gasLimit
4 participants