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

feat: Update Avnu sdk #208

Open
RedBeardEth opened this issue Apr 21, 2024 · 2 comments
Open

feat: Update Avnu sdk #208

RedBeardEth opened this issue Apr 21, 2024 · 2 comments
Assignees
Labels
good first issue Good for newcomers ODHack

Comments

@RedBeardEth
Copy link
Contributor

Describe the feature you'd like to request

v2 of Avnu sdk (https://github.com/avnu-labs/avnu-sdk/releases/tag/v2.0.0) has been released enabling gasless transactions

Describe the solution you'd like to see

Upgrade Avnu and incorporate new functionality (e.g gasless if possible)

Additional information

No response

@RedBeardEth RedBeardEth added good first issue Good for newcomers ODHack labels Apr 21, 2024
@brolag
Copy link
Contributor

brolag commented Apr 21, 2024

Hi @RedBeardEth I'd like to work on this.

@brolag
Copy link
Contributor

brolag commented Apr 26, 2024

Update:

Hi @RedBeardEth,

I'm testing the swap on Sepolia using the STRK contract instead of LORDS
0x04718f5a0fc34cc1af16a1cdee98ffb20c31f5cd61d6ab07201858f4287c938d

image

Given the gasless feature supports USDC or USDT I'm using USDC. Here's my config:

image

According to the answers from the AVNU support team having USDC in the wallet is required for the gasless feature. In the AVNU swap it's pretty clear that there are two options:

image

Based on that I have a few questions:

  1. In this moment is not clear for the user what's happening (even for the AVNU fi swap). I think is a good idea to handle this exception somehow.

  2. There's not enough liquidity for USDC, is there a way to get USDC tokens easly on Sepolia so I can test better?

  3. The library update is working without the gasless options. Should I create a PR for this part only and tackle the gasless feature in a separate issue?

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers ODHack
Projects
None yet
Development

No branches or pull requests

2 participants