Upgrade UI - Screen Transactions

[saltict]

Update Background

• [x] Handle Transaction Queue
• [x] Handle Confirmation Queue
• [x] Handle Signature
• [x] Upgrade message for transaction

  sequenceDiagram
  UI->>+TransactionHandler: Create Transaction
  TransactionHandler->>+RequestHandler: Create Request
  RequestHandler->>-UI: Require Confirm
  UI->>+Keyring: Sign Transaction
  Keyring->>-UI: Signature
  UI->>+TransactionHandler: Send (Signature + Transaction)
  TransactionHandler->>-TransactionHandler: Do Transaction + Update RS
  TransactionHandler->>+UI: Notify RS

Implement API

• [x] Transaction Screens
  • [x] Transfer + XCM
  • [x] Send NFT
  • [x] Transaction
  • [x] Stake
  • [x] Unstake
  • [x] Withdraw
  • [x] Claim Reward

[sokol142]

Version test: https://file.cdmteck.com/s/4oimk9zDJ8R5Aai/download?path=%2F&files=./upgrade-ui-build-0.8.1-1679298208346.zip

Please check issues: General

• [x] Still showing error message when user click on back button

• [x] Handle case don't get transaction's fee

• Actual: Do not send token

• Expected: Allow send token with estimated fee = 0

• [x] Handle case check validate of the form

  Send token

• [x] Do not send some token

---

Send NFT

• [x] Do not send NFT (Moonbase's NFT)

• [x] Do not validate "Recipient" in case Scan QR Steps:

  1. Go to the Send NFT (Example: Send NFT on the Kusama chain)
  2. On the Send NFT, click on Scan QR code
  3. Scan the EVM address
  4. Click on Next button and observe the screen displayed

Actual: Do not validate, still redirect user to the "Confirmation" screen & send NFT successful Expect: When click on "Next" button -> show error

• [x] Router bug Steps:

  1. Go to the Send NFT screen
  2. Input data for all fields then click on Next button
  3. Click on Cancel button to go to back the Send NFT screen
  4. Reload page and observe the screen displayed

  Actual: Router error

[sokol142]

Version test: https://drive.google.com/file/d/1cmBsZ4Iy2bs4O-YxlkAphe9jKmQMZG8p/view?usp=share_link

Send token

• [x] Do not showing transaction history in case:

  • Send token with substrate account
  • Send token with EVM account (same wallet): do not showing history of the recipient

• [x] Recheck action go to back the Transfer screen when submitting

• [x] Showing duplicate error message

• [x] Update style of the Amount field

  • Do not allow input "-" character

• [x] Show incorrect Transferable balance when send token from "All accounts"

• [x] Handle case switch account when user open both popup view & expand view Steps:

  1. Stand on any substrate account with Expand mode
  2. Go to the Send token screen
  3. Open the popup mode, switch to any EVM account
  4. Observe the screen displayed on the expand mode

  Actual: Show the Token & Chain incorrect Expect: When switch account -> Back to Homepage (current Tab)

• [x] Do not verify the same account when the user uses the network address Example: This account have:

  • Substrate address: 5HbcGs2QXVAc6Q6eoTzLYNAJWpN17AkCFRLnWDaHCiGYXvNc
  • Polkadot address: 16XuRCHUPGS5Xw7Am73LgWzTNSMeoUJLKv5GfWZdkoJ4iVZc Steps:
    1. Switch to account (5HbcGs2QXVAc6Q6eoTzLYNAJWpN17AkCFRLnWDaHCiGYXvNc)
    2. Go to the Send fund screen
    3. On the Send fund screen, input "Send to account" = 16XuRCHUPGS5Xw7Am73LgWzTNSMeoUJLKv5GfWZdkoJ4iVZc
    4. Click on "Transfer" button

  Actual: Do not verify the same account

  Expected: Verify the same account -> Highlight "send to account" field and show error message

• [x] Do not perform xcm

• [x] Validate incorrect in case xcm transfer (substrate to evm)

---

Notes for tester

• Test cases send token with local token later

[sokol142]

Version test: https://file.cdmteck.com/s/4oimk9zDJ8R5Aai/download?path=%2F&files=./upgrade-ui-build-0.8.1-1679550205800.zip

• [x] Still shows incorrectly Transferable balance when standing on All accounts

• [x] Hide the Send button with a read-only account => Solution: Show message to warning to user

---

Send token

• [x] Do not send local token
  • Example: Send GLMR (Acala)

[sokol142]

• [x] UI bug

[sokol142]

• [x] Do not send ERC-20 token
• Reproduce with MFG (Moonbase Alpha) has contract address: 0xb161b2da48de283ec22bafbc36e5551c892629a2

[sokol142]

• [x] Validate incorrect List account when the user sends a token from the Token Detail screen with All accounts mode
  • Actual:
    • Show list account incorrect
    • Do not show token default if have only 1 token on the Token detail view
  • Expect: View detail in BA Document: https://www.notion.so/subwallet/Feature_Manage-token-6bcf46a6bd004d64b52a0ce964c94929?pvs=4#be8fc93a8eda4447906bf2dce344a2e6

[sokol142]

• [x] An error occurred when the user click on the Back button on the "Transaction Result" screen after Send NFT successful

[sokol142]

• [x] Show require message when clicking on the Max button

  • Steps:

    • Do not enter Amount
    • Out focus -> Show require message
    • Click Max

  • Actual: Still show require message

  • Expect: Do not show a message

[sokol142]

• [x] Show incorrect token list
  • Actual: Show ERC-721 (custom NFT)

[sokol142]

• [x] Do not send local tokens on the EVM account
  • Reproduce the behavior with xcACA (Moonbeam)

[sokol142]

Version test: https://file.cdmteck.com/s/4oimk9zDJ8R5Aai/download?path=%2F&files=./1122-merge-build-1.0.1-1680512044695.zip

• [x] An error occurs in case sending a token with a Ledger account when the user standing on the Token Detail screen