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

Fix issue with output token decimals #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cryptopavelsan
Copy link

The output token amount from the swap was using the input token's decimals. Would work of if they matches but fails for tokens like USDC to SOL swaps. This fixes it to use the correct decimal number for the correct display of the swap amount on screen to the user.

The output token amount from the swap was using the input token's decimals. Would work of if they matches but fails for tokens like USDC to SOL swaps. This fixes it to use the correct decimal number for the correct display of the swap amount on screen to the user.
arbcaptain pushed a commit that referenced this pull request Dec 12, 2022
arbcaptain pushed a commit that referenced this pull request Dec 13, 2022
Added in the new transaction retry logic (v1) that simply pauses, fetches a new route list and tries again (top route) up to the max set in the .env variable. Mixing this logic with using existing route #1 and #2 may be a good middle ground. Also added some styling and a few more bonuses here to make it better. Also limited the Jupiter top result to 1 to add some efficiency with the list.
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.

1 participant