status-im / status-desktop

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

[Wallet] Watched address assets being shown in other addresses #14769

Open benjthayer opened 2 months ago

benjthayer commented 2 months ago

The assets from watched addresses are currently being shown in other addresses.

For example, I have added this watched address which has (or has had in the past) balances of MATIC, WAMPL, and WETH. Those assets are showing in the watched address asset tab even if the current balance is zero - which is the correct behaviour.

However, those assets are also appearing in my other accounts which have never had balances of those tokens - which is the incorrect behaviour. Each individual account should only show assets which that account currently hodls or has hodl'd in the past.

The watched address tokens should only appear in the assets tab of the watched address (and the All accounts assets tab if the user has opted to include that watched address' balances and activity).

Watched address containing the assets MATIC, WAMPL and WETH:

Screenshot 2024-05-15 at 12 04 52

Status created account incorrectly showing those same assets:

Screenshot 2024-05-15 at 12 05 26

cc @Khushboo-dev-cpp

saledjenic commented 1 month ago

Those assets are showing in the watched address asset tab even if the current balance is zero - which is the correct behaviour.

@benjthayer why do you think it's ok to display tokens with 0 balances in the assets tab?

benjthayer commented 1 month ago

Those assets are showing in the watched address asset tab even if the current balance is zero - which is the correct behaviour.

@benjthayer why do you think it's ok to display tokens with 0 balances in the assets tab?

It's quite common in Wallets to display the assets the user has previously held when their current balance is 0. We don't want to show all assets in the token list, just those that the user has held and previously sent/spent. The user can manually hide these if they want to.

There's a few benefits to this: