Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Restoring the same wallet a few times doesn't restore all NFTs in one go #8744

Open
srirambv opened this issue Feb 7, 2024 · 1 comment
Open

Comments

@srirambv
Copy link
Contributor

srirambv commented Feb 7, 2024

Description:

Restoring the same wallet a few times doesn't restore all NFTs in one go

Steps to Reproduce

  1. Restore wallet with NFTs
  2. Go to NFT tab and enable auto-discovery
  3. Ensure all NFTs are restored
  4. Reset wallet and restore same wallet from step 1
  5. Go to NFT tab and wait for auto discovery
  6. Only few NFTs are restored
  7. Lock and unlock or close and open wallet triggering fetch
  8. May or may not restore all NFT. Need to repeat step 7 until all of them are restored

Actual result:

Restoring the same wallet a few times doesn't restore all NFTs in one go

Expected result:

Should restore all NFTs every time the wallet is restored

Reproduces how often: [Easily reproduced, Intermittent Issue]

Easy

Brave Version:

  • Can you reproduce this issue with the most recent build from TestFlight? Yes
  • Can you reproduce this issue with the previous version of the current build from TestFlight? Yes
  • Can you reproduce this issue with the current build from AppStore? Yes

Device details:

All

Website problems only:

  • did you check with Brave Shields down? NA
  • did you check in Safari/Firefox (WkWebView-based browsers)? NA

Additional Information

cc: @nuo-xu @StephenHeaps

@nuo-xu
Copy link
Contributor

nuo-xu commented Feb 8, 2024

@nvonpentz i did some tests. I think this could be related to brave/brave-browser#35697. However, this time, we can trigger asset discovery by calling WalletService.discoverAssetsOnAllSupportedChains but core only returns some assets not all assets via WalletService.onDiscoverAssetsCompleted callback.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Status: Backlog
Development

No branches or pull requests

2 participants