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

!WIP: fix: improve TON signer and MyTonWallet provider #914

Draft
wants to merge 2 commits into
base: next
Choose a base branch
from

Conversation

Ikari-Shinji-re
Copy link
Member

Summary

  • Fix the issue where TON tokens are not being displayed correctly in the token list
  • Return the non-bounceable address for the MytonWallet provider
  • Retrieve and return the hash of the message from the BOC after signing the TON transaction

How did you test this change?

  • Review the list of TON tokens in the token list. Ensure that if a token has an address or name, it is displayed correctly. You can verify this by checking the meta response from the network tab and comparing it with the UI.
  • Connect the MytonWallet provider. The address displayed in the confirmation wallet modal must match the address selected and shown in the MytonWallet UI.
  • Create a transaction using the TON blockchain as the source. Sign the transaction and check the network tab to ensure that the hash of the message is sent to the server with the checkStatus request.

Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • Implemented a user interface (UI) change, referencing our Figma design to ensure pixel-perfect precision.

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