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

GSW-530 feat: swap router #81

Merged
merged 35 commits into from
Nov 7, 2023
Merged

GSW-530 feat: swap router #81

merged 35 commits into from
Nov 7, 2023

Conversation

r3v4s
Copy link
Member

@r3v4s r3v4s commented Nov 6, 2023

  • BestSwap() handles all type of swap (exactIn / exactOut), (singlePath, multiPath), (singlePool, multiPool)
  • BestSwapDry() will return best route path depends on input paramter
  • tokenPrice now will be updated with base token gno.land/r/foo

BREAKING CHANGE

  • pool.Swap() will be deprecated next version, call router.BestSwap()

@r3v4s r3v4s requested a review from notJoon November 6, 2023 11:05
@r3v4s r3v4s self-assigned this Nov 6, 2023
@r3v4s r3v4s force-pushed the GSW-530-feat-swap-router branch from c0e8b38 to 8a4592c Compare November 6, 2023 11:06
Copy link
Member

@notJoon notJoon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work 👍. I've touched on a few things, but just take it as a guide, and please point out any errors.

router/find_path.gno Outdated Show resolved Hide resolved
router/getter_api.gno Outdated Show resolved Hide resolved
router/getter_api.gno Outdated Show resolved Hide resolved
router/getter_api.gno Outdated Show resolved Hide resolved
router/quotation.gno Outdated Show resolved Hide resolved
router/quotation.gno Show resolved Hide resolved
router/quotation.gno Show resolved Hide resolved
@r3v4s r3v4s requested a review from notJoon November 7, 2023 03:56
@r3v4s
Copy link
Member Author

r3v4s commented Nov 7, 2023

@notJoon bit of err found which I left in comments, and made function in getter_api recursive too as quotation

@r3v4s r3v4s merged commit 030936c into main Nov 7, 2023
@notJoon notJoon deleted the GSW-530-feat-swap-router branch November 7, 2023 08:36
@r3v4s r3v4s restored the GSW-530-feat-swap-router branch November 7, 2023 09:34
@r3v4s
Copy link
Member Author

r3v4s commented Nov 7, 2023

@mconcat branch is merged for further PR, but pls leave your comments :D

@r3v4s r3v4s deleted the GSW-530-feat-swap-router branch November 22, 2023 09:43
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