8000 atomic swaps [APP-2658] by derHowie · Pull Request #6695 · rainbow-me/rainbow · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from
Open

atomic swaps [APP-2658] #6695

wants to merge 6 commits into from

Conversation

derHowie
Copy link
Member
@derHowie derHowie commented Jun 17, 2025

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.

return shouldDelegate;
};

export const getApproveAndSwapCalls = async (quote: Quote | CrosschainQuote) => {
Copy link
Member Author

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) => {
Copy link
Member Author
@derHowie derHowie Jun 17, 2025

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

@derHowie derHowie marked this pull request as ready for review June 20, 2025 15:39
@derHowie derHowie requested a review from jinchung June 20, 2025 15:39
@derHowie derHowie changed the title atomic swaps atomic swaps [APP-2658] Jun 20, 2025
Copy link
linear bot commented Jun 20, 2025

@derHowie derHowie force-pushed the @chris/atomic-swaps branch from be9885d to cb6aba8 Compare June 20, 2025 15:48
Copy link
socket-security bot commented Jun 20, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatednpm/​isows@​1.0.3 ⏵ 1.0.71001007587100
Updatednpm/​abitype@​0.9.8 ⏵ 1.0.8100 +1100100 +184100
Addednpm/​@​noble/​ciphers@​1.3.010010010088100
Updatednpm/​ox@​0.1.2 ⏵ 0.7.1100 +110099 +197100
Updatednpm/​viem@​1.21.4 ⏵ 2.31.0100 +610010097100

View full report

@jinchung
Copy link
Member

Launch in simulator or device for ad72b5a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0