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/ldg 651 implement method redeem #7

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

Steven2505
Copy link

Checklist

  • App update process has been followed
  • Target branch is develop
  • Application version has been bumped

Example of Tx : https://80000.testnet.routescan.io/tx/0x96698f64a405fc1004894f59172ce930703f5532bd774a8d50af48d9bcca62e1

Possible de confirmer si je gère correctement l'amount ?

@Steven2505 Steven2505 requested a review from GuilaneDen January 22, 2025 09:04
@Steven2505
Copy link
Author

@GuilaneDen est-ce que tout te semble bon pour l'amount ? J'ai un doute sur si je le display correctement. Ou si le test case est même bon.

@GuilaneDen
Copy link

@Steven2505 De ce que je vois il y a 4 params. Le param Asset correspond au token utilisé ici c'est "015fd589f4f1a33ce4487e12714e1b15129c9329" qui correspond au token "USDC". Le param honeyAmount est l'amount à afficher. Le param receiver est le beneficiary et enfin le boolean.

Pour résumer l'écran d'amount devrai être "USDC 2" pour moi.

@Steven2505
Copy link
Author

@GuilaneDen Du coup la c'est pas bon alors, il faudrait que dans mon parsing reprennes le params asset pour donner le bon ticker au params amount si j'ai bien compris ?

@GuilaneDen
Copy link

@Steven2505 Oui il faut que tu utilises le fichier handle_provide_token pour ça.

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