-
Notifications
You must be signed in to change notification settings - Fork 608
governance-bridge-simulation #1467
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
Closed
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* add TempOwner contract mixin This mixin adds an extension to be able to execute a function from a hardcoded address with an EOL * add OwnerRelayOnOptimism.directRelay method This method can only be executed by the temporary owner before the specified EOL date on contructor * refactor relayCall fn declaration * fix error messages length to be <32bytes * add TempOwned tests * add debugging methods for now variable * update tests for initial failing states * update relay finish event to relayed call only * add tests using fastForward to update current timestamp * add EOL reach test case for OwnerRelayOnEthereum.directRelay * fix test title typo * Rename internal functions with underscore * Add require checks to temp owner * Improvements on TempOwned tests * Improvements to directRelay() tests * Update integration tests for owner relays with directRelay() * Renamed TempOwned to TemporarilyOwned and changed time mechanism * Tweaks to TemporarilyOwned.sol Co-authored-by: Alejandro <palebluedot@gmail.com>
…idge (#1466) * Remove interface casting * Remove functionality for accepting ownership directly on L2 * Add relay batch support and unit tests * Make directRelay() integration tests L2 scope only * Modify dual relay integration tests to relay a single L1-L2 interaction * Add relayBatch integration tests * Renams data to payload(s) * Dedup xGasLimit calculation * Dedup msg and xDomain sender checks * Rename OwnerRelayOnOptimism events * Address new PR comments
* Scripts updated * Some fixes * Extracted some utils * remove comment * Some fixes
…into governance-bridge-simulation
…/synthetix into governance-bridge-simulation
unused |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Governance Bridge Simulation - Full System fresh deployed in Kovan