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

Update swap examples #81

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Update swap examples #81

wants to merge 4 commits into from

Conversation

tatata96
Copy link

@tatata96 tatata96 commented Feb 7, 2024

No description provided.

@tatata96 tatata96 added the bug Something isn't working label Feb 7, 2024
@tatata96 tatata96 self-assigned this Feb 7, 2024
@tatata96 tatata96 removed the bug Something isn't working label Feb 7, 2024
@tatata96 tatata96 requested a review from Anlerkan February 7, 2024 06:02
@Anlerkan Anlerkan requested a review from gulcinuras February 7, 2024 10:17
@Anlerkan Anlerkan changed the title Update swap examlpes Update swap examples Feb 7, 2024
Copy link
Contributor

@Anlerkan Anlerkan left a comment

Choose a reason for hiding this comment

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

Looks like the files are not correctly formatted. Can you please fix it @tatata96 ?
image

examples/src/index.ts Outdated Show resolved Hide resolved
examples/src/operation/bootstrap/bootstrapPool.ts Outdated Show resolved Hide resolved
@@ -30,42 +30,33 @@ export async function fixedOutputSwap({
* Swap.getQuote method, which will return the best quote (highest rate)
* after checking both v1 and v2
*/
const fixedOutputSwapQuote = Swap.v2.getQuote(
SwapType.FixedOutput,
const fixedOutputSwapQuote = await Swap.v2.getQuote({
Copy link
Contributor

@Anlerkan Anlerkan Feb 20, 2024

Choose a reason for hiding this comment

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

These examples use the swap router by default. Can you please add an example that does not use the swap router for both fixedInputSwap and fixedOutputSwap.

Copy link
Contributor

@gulcinuras gulcinuras left a comment

Choose a reason for hiding this comment

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

Hey @tatata96, some of the files seem unformatted. If you can format them all, would be great

examples/src/index.ts Outdated Show resolved Hide resolved
“Tamara added 2 commits March 7, 2024 13:36
add condition for addInitialLiquidtiy
add example without swap router to fixedOutputSwap
format files
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.

3 participants