Skip to content
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

chore: swap refactor #3731

Merged
merged 71 commits into from
Jan 27, 2025
Merged

chore: swap refactor #3731

merged 71 commits into from
Jan 27, 2025

Conversation

jorbuedo
Copy link
Contributor

@jorbuedo jorbuedo commented Nov 11, 2024

Description / Change(s) / Related issue(s)

Work in progress, don't commit to it

  • Swap Api types
  • Dexhunter endpoints and transforms
  • Muesliswap endpoints and transforms New Muesliswap endpoints and transforms
  • Muesliswap calculations
  • Muesliswap tx creation Leaks aggregator difference, so the client needs to either get a cbor from dexhunter, or the datum/hash/address from muesliswap to construct its own txNew endpoint comes with cbor, no need to create it
  • Swap Manager types
  • Swap manager
  • Swap mobile client
  • Fix decimals difference between aggregators and any other inconsistencies found while running NEW: asked muesli for a new param on their endpoints, to request and receive amounts with decimals baked in, so this issue will be solved with that.
  • Dexhunter Sign endpoint
  • First full run attempt
  • Product changes to adapt to multiple aggregators, specially but not limited to Splits (multiple trades instead of only a single 1)
  • Add 100% tests coverage

@stackchain stackchain changed the title WIP: Swap refactor chore: swap refactor Jan 20, 2025
@stackchain stackchain added dont-merge wip Shows that a PR shouldn't be merge labels Jan 20, 2025
@stackchain stackchain added this to the 5.5.0 milestone Jan 20, 2025
@stackchain stackchain marked this pull request as ready for review January 20, 2025 14:20
@stackchain stackchain changed the base branch from develop to refactor/swap-module January 27, 2025 11:46
@stackchain stackchain merged commit 197b35c into refactor/swap-module Jan 27, 2025
1 check passed
@stackchain stackchain deleted the YV-47-alt branch January 27, 2025 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dont-merge wip Shows that a PR shouldn't be merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants