status-im / status-desktop

Status Desktop client made in Nim & QML
https://status.app
Mozilla Public License 2.0
292 stars 78 forks source link

When purchasing stickers user should be able to select any account from the dropdown list #16404

Open virginiabalducci opened 3 weeks ago

virginiabalducci commented 3 weeks ago

Bug Report

Description

if I have default wallet account selected or all accounts selected in the wallet sectiono, and I have one other account imported with funds and I want to buy stickers for example, always the selected account in the wallet section is selected as default one in the send modal popup.

Steps to reproduce

  1. remove local Status folder
  2. create a new Status profile
  3. default account will be named Account 1
  4. then import seed phrase that has account with some funds
  5. then go to the chat (but for this you need to add a contact or join a community)
  6. and try to buy sticker (any)
  7. you will see a send modal with Account 1 as the selected address, if you try to change it you won't be able to do that cause it will. be always resetted to Account 1
  8. then if you go to the wallet section and select that imported account, then go back to chat and try to buy sticker, that added account will be the selected one

Expected behavior

Actual behavior

Additional Information

This issue was reported by @saledjenic I will add my testing results on the next comment

virginiabalducci commented 3 weeks ago

Test 1: I imported two accounts to a new Status profile account, but in both cases I'm able to switch between accounts when on the modal to purchase stickers.

https://github.com/user-attachments/assets/16babed6-d86e-41d8-82db-c8ad5f0a830b

Test 2: I was however, able to reproduce something similar where the account selected on the Wallet section cannot be changed on the modal to buy stickers. It seems this is related to the account having enough SNT to pay for the stickers, whereas on the first video, the accounts have funds but no SNT tokens.

https://github.com/user-attachments/assets/b1945e8a-a424-402e-96f3-9084223f5afa

I'm attaching the logs from the account used to test this second scenario app_20240925_130241.log app_20240925_132055.log

saledjenic commented 2 weeks ago

@virginiabalducci please check the video and you will see how it behaves for me. The default selection follows the selected account in the Wallet section, that's ok, but I cannot change the selected account when running the buying ens name flow or buying stickers flow.

https://github.com/user-attachments/assets/bd61f94f-1f0e-4234-9341-65886eaa6d5a

virginiabalducci commented 2 weeks ago

@saledjenic I was able to reproduce this issue on a master build of 1st October https://github.com/user-attachments/assets/1a0dd939-5019-40ea-bc1d-396e44de6684