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

exclude_dexes and priority_fee #9

Open
cryptoraptor opened this issue Mar 2, 2024 · 3 comments
Open

exclude_dexes and priority_fee #9

cryptoraptor opened this issue Mar 2, 2024 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@cryptoraptor
Copy link

Hey, great work on SDK!

There is issue with exclude_dexes in jupiter swap function
quote_url += "&excludeDexes=" + ','.join(exclude_dexes).lower -> quote_url += "&excludeDexes=" + ','.join(exclude_dexes)

remove .lower because the query fails

also priority_fee needs to be added to swap Line:645
priority_fee: int=0

and here Line:690
transaction_parameters = {
"quoteResponse": quoteResponse,
"prioritizationFeeLamports":priority_fee,
"userPublicKey": user_public_key,
"wrapAndUnwrapSol": wrap_unwrap_sol
}

and again thanks for great job!

@0xTaoDev 0xTaoDev self-assigned this Mar 2, 2024
@0xTaoDev 0xTaoDev added the bug Something isn't working label Mar 2, 2024
@0xTaoDev
Copy link
Owner

0xTaoDev commented Mar 2, 2024

Hello, thanks a lot for finding those issues, very great! I'll fix that tomorrow, thanks a lot for your contribution

@techbro111
Copy link

techbro111 commented Mar 23, 2024

Can the priority fee be added to the DCA method? Im getting a lot of failed tx

@0xTaoDev
Copy link
Owner

0xTaoDev commented Apr 8, 2024

Can the priority fee be added to the DCA method? Im getting a lot of failed tx

I didn't find any information about setting the priority fee for DCA, did you find any solution to this problem?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants