-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Deploy ERC-1820 to Kovan and Mainnet #1883
Comments
Yep, working on it! |
cc @tynes @mslipper we're going to have to get rid of the upper bound allowable gas price for this to work. I don't really know why that upper bound exists in the first place, so I don't see the problem in doing so. I believe this upper bound is only on the Sequencer side so should be an easy upgrade to execute. |
The bound exists to prevent users from accidentally sending transactions with too high of a fee. We can remove it if its causing other issues. |
Should be deployed to Kovan by EOD today. Mainnet tomorrow, I think. |
EIP-1820 has now been deployed to Kovan: https://kovan-optimistic.etherscan.io/address/0x1820a4b7618bde71dce8cdc73aab6c95905fad24 |
EIP-2470 has now been deployed to Kovan: https://kovan-optimistic.etherscan.io/address/0xce0042b868300000d44a59004da54a005ffdcf9f |
thanks! |
I've funded both deployer accounts, now just waiting for the addresses to be whitelisted so I can execute the transactions on mainnet. Hopefully that'll be done by the end of the day today. |
EIP-1820 and EIP-2470 have now been deployed to OE mainnet! Closing this issue. EIP-1820: https://optimistic.etherscan.io/address/0x1820a4b7618bde71dce8cdc73aab6c95905fad24#code |
TL;DR: please execute
Context:
This deploys ERC-1820, the lack of which has been blocking me for weeks now.
ERC-1820 is a registry contract widely used on mainnet and other networks. A range of other EIPs - e.g. ERC-777 - depend on it.
It was a predeployed contract on OVM 1.0.
On OVM 2.0, it's not predeployed anymore, because - in theory - it can now be deployed at runtime.
In practice, I couldn't deploy it myself because
a) the sequencer blocked unprotected transactions. This is now fixed with #1720
b) the sequencer rejects the tx above with
Error: gas price too high: 100000000000 wei, use at most tx.gasPrice = 10000 wei
Since I don't have an issue with that limitation, but just need ERC-1820, please somebody from the team deploy it.
The text was updated successfully, but these errors were encountered: