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

Mobile - Unified account - Implementing validate for transfer TON feature #1858

Closed
3 tasks done
Sokol142196 opened this issue Dec 16, 2024 · 3 comments
Closed
3 tasks done
Assignees
Milestone

Comments

@Sokol142196
Copy link

Sokol142196 commented Dec 16, 2024

Describe the task
Implementing validate for transfer TON feature

  • Add warning in case transfer to bounceable + inactive address
  • Allow transfer same address (sender = receiver)
  • Re-check input address for TON and TON testnet on transfer feature
    • Expect:
      • Distinction between mainnet and testnet addresses
      • Synchronize addresses in all steps: Input, Transfer Confirmation, History
@giangtth17
Copy link
Collaborator

giangtth17 commented Dec 24, 2024

Version: SubWallet v1.1.84(440)b-v11
Device: Samsung S20

  • Add warning in case transfer to bounceable + inactive address
    Actual: Didn't show warning in case transfer to bounceable + inactive address

    Screen_Recording_20241224_162634_SubWallet.mp4

    Expect: Add warning in case transfer to bounceable + inactive address

    image

  • Check to get transaction information in the History screen
    Reproduce the steps:
    1. Make valid transfer steps with TON account
    2. Check transaction information in the history screen
    Actual:
    - At the history screen, the transaction status message is displayed as "Failed"
    - At the Token detail screen of TON shows that the token has been successfully traded
    - Check on the website
    [https://testnet.tonviewer.com/transaction/58372a48a725bad1a554b558e9fbd21179c86a9b4cbe23e3ce762b3bf92c2f97] that the transaction has been completed successfully
    Expect: Display correct transaction information
    Attach file:
    Image

    image

@ThaoNguyen998
Copy link

ThaoNguyen998 commented Dec 30, 2024

Version test:
Image

  • Do not change version TON mainnet (Solo account TON)

  • Update logo account on history detail transfer TON
    Actual: Do not show logo

    Image

    Expect: Show logo

  • Do not get status on History (Check rpc) -> Follow up

@ThaoNguyen998
Copy link

Version test:
Image
Device test: IP 13

Validation passed.

@ThaoNguyen998 ThaoNguyen998 added this to the Milestone 8 milestone Dec 30, 2024
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

No branches or pull requests

4 participants