-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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: Add gas included swaps #13430
feat: Add gas included swaps #13430
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
Updated dependencies detected. Learn more about Socket for GitHub ↗︎
|
24f293e
to
2d94ae8
Compare
|
|
38aa624
to
67c1335
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #13430 +/- ##
==========================================
- Coverage 61.83% 61.82% -0.01%
==========================================
Files 1975 1975
Lines 43847 43889 +42
Branches 5926 5929 +3
==========================================
+ Hits 27111 27136 +25
- Misses 14935 14952 +17
Partials 1801 1801 ☔ View full report in Codecov by Sentry. |
|
|
014d439
to
e24eb98
Compare
|
The PR has been merged in order to make it into a release candidate today. There is one known issue which will be resolved in a follow-up PR asap and then cherry picked. |
Description
Users who don't have enough ETH to pay for gas fees while swapping will be able to swap now if the gas fee can be taken from one of the tokens they are swapping. Smart transactions must be enabled for this.
Related issues
Fixes:
Manual testing steps
Test case 1:
Test case 2:
Use max
for ETHScreenshots/Recordings
User doesn't have enough ETH for gas fees (gas included swaps):

User has enough ETH for gas fees:

Before
After
Pre-merge author checklist
Pre-merge reviewer checklist