Skip to content

[Bug]: NFTs owned by imported account is shown on wallet first account after importing #9196

@chrisleewilcox

Description

@chrisleewilcox

Describe the bug

NFTs owned by an imported account (oragefox.eth) are showing for the first wallet account. This is happening for both upgrading app or new installs.

Before upgrading from v7.19.0 to v7.20.0 my wallet had 3 wallet accounts, 1 imported account (orangefox.the), and 2 hw accounts. The imported account owned multiple NFTs on Ethereum Mainnet and all other accounts did not own any NFTs on Ethereum Mainnet. NFTs were properly being shown only for the imported account on the NFTs tab.

After upgrading to v7.20.0 I now see some NFTs owned by the imported account showing on the first wallet account (Account 1).

The issue does not occur instantly after importing or upgrading. Issue started showing soon after importing and then switching accounts.

This is also happening for new installs and importing an account that owns NFTs.
This is also happening for both iOS and Android devices.

Not able to reproduce on v7.19.0.

Expected behavior

NFTs should only be displayed for the selected account that owns NFTs.

Screenshots/Recordings

Screen.Recording.2024-04-10.at.9.16.13.AM.mov

Steps to reproduce

GIVEN I installed MMM
AND I have completed onboarding
AND wallet is backed up
AND Ethereum Mainnet is selected
WHEN I import an account that owns NFTs
AND I enable Autodetect NFTs
THEN NFTs are displayed on NFTs view in main wallet view for imported account
WHEN I switch to the wallets first account
AND first account does not own NFTs
THEN no NFTs are displayed for first account (FAIL: NFTs from imported account are shown for first account)
AND NFTs are not displayed for any other account that doesn't own NFTs

Error messages or log output

No response

Version

7.20.0

Build type

None

Device

Any

Operating system

iOS, Android

Additional context

No response

Severity

Sev1

Metadata

Metadata

Assignees

Labels

NFTsissues related to collectibles - ERC721, ERC1155Sev1-highAn issue that may have caused fund loss or access to wallet in the past & may still be ongoingregression-RCDEPRECATED: Please use "regresssion-RC-x.y.z" label insteadregression-RC-7.20.0release-7.20.0Issue or pull request that will be included in release 7.20.0release-blockerThis bug is blocking the next releaseteam-assetstype-bugSomething isn't working

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions