-
Notifications
You must be signed in to change notification settings - Fork 47
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
feat(extension): Add OptimisticAuctionRebalanceExtension #156
feat(extension): Add OptimisticAuctionRebalanceExtension #156
Conversation
…fix compilation issues
test/integration/ethereum/optimisticAuctionRebalanceExtenisonV1.spec.ts
Outdated
Show resolved
Hide resolved
test/integration/ethereum/optimisticAuctionRebalanceExtenisonV1.spec.ts
Outdated
Show resolved
Hide resolved
* Add events for updating the isOpen parameter * lintoor * remove .only() * lintoor * fix AssetAllowList nits * Run only new integration tests * Fix unittests * Add tests for require statements in proposeRebalance * Add tests for emitted events * Extend event emission test * Adjust integration test to use index token as proposer collateral (currently failing) * integration test test * cleanups * edit integration tests * Test bond transfer and claim construction * Fix unittest --------- Co-authored-by: ckoopmann <[email protected]>
…thub.com:IndexCoop/index-coop-smart-contracts into ckoopmann/optimistic-auction-rebalance-extension
… of _constructClaim
…tps://github.com/IndexCoop/index-coop-smart-contracts into ckoopmann/optimistic-auction-rebalance-extension
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.
LGTM 💯
We may want to peek the flaky optimism integration test
Seems to be an issue on alchemy side for the optimism rpc. |
No description provided.