-
Notifications
You must be signed in to change notification settings - Fork 197
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: [asset-swapper] Add Crypto.com as a source #43
Conversation
case ERC20BridgeSource.CryptoCom: | ||
// HACK: deploy using unique address | ||
return opts.contractAddresses.sushiswapBridge; |
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 the only side effect of this that it would show up as sushi volume in analytics?
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.
Yeah and avoid paying the overhead of FQT<-->Bridge transfer costs
6a296bd
to
2c924bb
Compare
9d134c9
to
4dbf7da
Compare
@@ -332,6 +332,7 @@ export async function runMigrationsAsync( | |||
dodoBridge: NULL_ADDRESS, | |||
dodoHelper: NULL_ADDRESS, | |||
snowSwapBridge: NULL_ADDRESS, | |||
cryptoComBridge: NULL_ADDRESS, |
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.
I thought we don't have bridges in this interface anymore?
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.
What do you mean, we need it to branch off in the BridgeAdapter
.
We took them out of contract-addresses
and moved them into AS.
85681df
to
6a96c93
Compare
Description
Adds Crypto.com as a source, a fork of Uniswap V2.
Sims
TODO:
eligible for VIPTesting instructions
Types of changes
Checklist:
[WIP]
if necessary.