-
Notifications
You must be signed in to change notification settings - Fork 666
atomic swaps [APP-2658] #6695
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
base: develop
Are you sure you want to change the base?
atomic swaps [APP-2658] #6695
Conversation
return shouldDelegate; | ||
}; | ||
|
||
export const getApproveAndSwapCalls = async (quote: Quote | CrosschainQuote) => { |
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 it okay to loadWallet
here? @jinchung
return delegateCalldata; | ||
}; | ||
|
||
export const simulateDelegatedTransaction = async (quote: Quote | CrosschainQuote) => { |
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.
Loading wallet above makes simulation / gas estimation easier to pass around
be9885d
to
cb6aba8
Compare
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Introduce viem options for provider and wallet:
loadWalletViem
+getProviderViem
.Incorporates the rainbow delegation package for transaction execution and for building gas estimation logic.
Will perform atomic batches of approve and swap when the experiment is switched on.
What to test:
Ensure that with the flag off, all swaps, whether they require an approval, perform as usual.
With the flag on, verify whether atomic batches of approve and swap are being executed.
Ensure with the flag on, swaps that do not require an approval still behave as normal.