Skip to content
This repository was archived by the owner on Sep 15, 2022. It is now read-only.

lower default gas price #22

Merged
merged 1 commit into from
Jan 8, 2020
Merged

lower default gas price #22

merged 1 commit into from
Jan 8, 2020

Conversation

DJViau
Copy link
Contributor

@DJViau DJViau commented Jan 8, 2020

Got walloped by this again on the Mod Badge project 😬

Changes the default gas price from 50 GWEI to 5 GWEI.

Copy link
Contributor

@alexanderatallah alexanderatallah left a comment

Choose a reason for hiding this comment

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

@DJViau when deploying contracts, i’ve found that low gwei amounts often fail - this is for minting right? shouldn’t we edit script/mint.js? or did you mean when deploying the contract? if 5 gwei worked, that sounds good. preemptively approving

@DJViau
Copy link
Contributor Author

DJViau commented Jan 8, 2020

did you mean when deploying the contract? if 5 gwei worked, that sounds good. preemptively approving

@alexanderatallah I accidentally deployed the original contract at 50 gwei, which cost ~0.14 ETH. I just tested at 5 gwei twice and it worked nicely. Cost ~0.01 ETH the first time and weirdly only ~0.001 ETH the second. Took less than a minute each time. ETH Gas Station is showing a 1 gwei standard today, but I think it's safe nonetheless.

Merging, but I'll keep my ear to the ground for issues from the community.

@DJViau DJViau merged commit 2733f6b into ProjectOpenSea:master Jan 8, 2020
@DJViau DJViau deleted the dan/10/01/lower-default-gas branch January 8, 2020 19:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants