-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Use OptimismPortal.FINALIZATION_PERIOD_SECONDS if L2OutputOracle.version is '1.0.0'. #5277
Conversation
Develop -> Master
|
✅ Deploy Preview for opstack-docs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
This PR has been added to the merge queue, and will be merged soon. |
This PR is next in line to be merged, and will be merged as soon as checks pass. |
1 similar comment
This PR is next in line to be merged, and will be merged as soon as checks pass. |
Description
@eth-optimism/sdk starting at 2.0.0 is assuming the
FINALIZATION_PERIOD_SECOND
is moved to L2OutputOracle, which is not the case for L2OutputOracle.version === '1.0.0'.Version 1.0.0 of the L2OutputOracle is still deployed at Goerli, and all SDK consumers depending on the
FINALIZATION_PERIOD_SECOND
are broken right now (e.g. the fault-detector). This PR makes the sdk work with both 1.0.0 and the latest version of smart contracts.Tests
No tests added. Manually ran
yarn start
in fault-detector to confirm this fixed the breakage.Happy add a test if necessary.
Invariants
N/A since this PR does not change critical path.