-
Notifications
You must be signed in to change notification settings - Fork 15
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
fix: take fee on to usdc #4801
fix: take fee on to usdc #4801
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## release/1.3 #4801 +/- ##
===========================================
Coverage 72% 72%
===========================================
Files 403 403
Lines 67348 67387 +39
Branches 67348 67387 +39
===========================================
+ Hits 48684 48729 +45
Misses 16320 16320
+ Partials 2344 2338 -6 ☔ View full report in Codecov by Sentry. |
let stable_amount = *stable_amount; | ||
|
||
if swap.to == STABLE_ASSET { | ||
swap.final_output = Some(stable_amount); |
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.
let stable_amount = *stable_amount; | |
if swap.to == STABLE_ASSET { | |
swap.final_output = Some(stable_amount); | |
if swap.to == STABLE_ASSET { | |
swap.final_output = Some(*stable_amount); |
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.
Does work, I tried it
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.
Just one thing, looks good (As in it looks like it will work, good is different)
* fix: take network fee from output amount when swapping to stable * chore: set value directly * chore: bump runtime version * chore: clippy * chore: cleaner
* origin/main: Arbitrum gas limit multiplier (#4781) fix: don't set code red on "agg-key set by gov-key" (#4813) fix: sign tx with correct key during rotation (#4794) chore: arbitrum witnesser permission (#4798) Solana (#4414) feat: try-runtime build step on dev ci (#4807) feat: optimistic build, streamlined ci-main (#4806) chore: log raw dispatch error (#4809) feat: allow for single binary CFE upgrades (#4634) fix: take fee on to usdc (#4801) (#4804) refactor: minor cleanup of retrier code and vault pallet (#4803) feat: cf_boost_pool_details rpc (#4780) # Conflicts: # Cargo.lock # state-chain/chains/src/sol.rs # state-chain/chains/src/sol/program_instructions.rs # state-chain/chains/src/sol/sol_tx_building_blocks.rs # state-chain/chains/src/sol/token_instructions.rs # state-chain/runtime/src/safe_mode.rs
No description provided.