-
Notifications
You must be signed in to change notification settings - Fork 10
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
[R4R]-[fee]feat: add gas-oracle module and set&update tokenRatio(ETH/MNT) GasPriceOracle.sol #41
Conversation
Ran 109 test suites: 531 tests passed😊, 0 failed😂, 0 skipped (531 total tests) |
Ran 109 test suites: 531 tests passed😊, 0 failed😂, 0 skipped (531 total tests) |
No other comment |
Ran 109 test suites: 531 tests passed😊, 0 failed😂, 0 skipped (531 total tests) |
ec30506
op-chain-ops/genesis/check.go
Outdated
// Slot 0x00 (0) is the slot of tokenRatio | ||
common.Hash{}: common.HexToHash("0x0000000000000000000000000000000000000000000000000000000000001194"), | ||
// Slot 0x01 (1) is the slot of owner | ||
common.Hash{31: 0x01}: common.HexToHash("0x000000000000000000000000bda5747bfd65f08deb54cb465eb87d40e51b197e"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For mainnet, do we need to update this value? Maybe we need to read gasOracleOwner
from deploy-config.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, after deployment, will check the on-chain data with configs.
Ran 109 test suites: 531 tests passed😊, 0 failed😂, 0 skipped (531 total tests) |
Ran 109 test suites: 531 tests passed😊, 0 failed😂, 0 skipped (531 total tests) |
Core changes:
tokenRatio
(ETH/MNT)gas-oracle
module0x4000000000000
first, will change laterReleated PR: