-
Notifications
You must be signed in to change notification settings - Fork 44
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
improve: swap all Ovm adapters (excl. Optimism/Base) w/ generic adapter #1824
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: bennett <bennett@umaproject.org>
Signed-off-by: bennett <bennett@umaproject.org>
Signed-off-by: bennett <bennett@umaproject.org>
@@ -62,7 +62,7 @@ export class BlastBridge extends BaseBridgeAdapter { | |||
const l2Bridge = this.getL2Bridge(); | |||
const events = await paginatedEventQuery( | |||
l2Bridge, | |||
l2Bridge.filters.ERC20BridgeFinalized(l1Token, undefined, fromAddress), | |||
l2Bridge.filters.ERC20BridgeFinalized(undefined, l1Token, fromAddress), |
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.
This was actually an issue where I assumed the blast bridge followed the same interface as the OpStackDefaultERC20Bridge finalization events, but actually the token address positions are swapped. I.e. the first position is the address of USDB and the second position is DAI on mainnet, not the other way around.
Signed-off-by: bennett <bennett@umaproject.org>
This is ready to go in. |
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 should probably wait until Monday to get it in so we can react to any regressions.
logger, | ||
SUPPORTED_TOKENS[MODE], | ||
spokePoolClients, | ||
filterMonitoredAddresses(MODE) | ||
bridges, |
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.
Is bridges
used anywhere else? Could maybe squash it?
bridges, | |
constructBridges(MODE), |
@bmzig Should we get this one in today? |
This is the first part of migrating the op-stack adapters. This moves Mode, Redstone, Zora, Blast, and Lisk to the generic adapter format, leaving only Optimism and Base as the only adapters still with the "legacy" structure.