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

Discover assets loading indicator is not removed #7433

Closed
StephenHeaps opened this issue May 10, 2023 · 1 comment · Fixed by #7434
Closed

Discover assets loading indicator is not removed #7433

StephenHeaps opened this issue May 10, 2023 · 1 comment · Fixed by #7434

Comments

@StephenHeaps
Copy link
Contributor

Description:

A certain case can cause the loading spinner shown on Portfolio when discovering assets is not removed correctly.

Steps to Reproduce

  1. Unlock your wallet
  2. Wait for loading spinner beside Assets header to disappear
  3. Lock your wallet & immediately unlock it again
  4. Observe loading spinner never stops until scrolling

Actual result:

Simulator.Screen.Recording.-.iPhone.14.Pro.-.2023-05-10.at.17.17.37.mp4

Expected result:

Loading indicator is eventually removed after discovery completes

Reproduces how often: [Easily reproduced, Intermittent Issue]

Brave Version:

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

Device details:

Website problems only:

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

Additional Information

@StephenHeaps StephenHeaps self-assigned this May 10, 2023
@StephenHeaps StephenHeaps added this to the 1.52 milestone May 10, 2023
nuo-xu pushed a commit that referenced this issue May 10, 2023
…oading (#7434)

Fix for displaying loading state for asset auto discovery when not discovering assets due to rate limiting
@srirambv
Copy link
Contributor

srirambv commented Jun 5, 2023

Verification passed on the following devices running 1.52 (23.6.1.16)

  • Verified steps from #7434
  • Verified asset loading indicator is not shown every time wallet is locked and unlocked
iPhone 7+ (iOS 15.4.8) iPhone 13 (iOS 16.5) iPad Pro (iOS 16.6 Beta)
7433-iPhone.7+.MP4
7433-iPhone.13.MP4
7433-iPad.Pro.MP4

arthuredelstein pushed a commit to brave/brave-core that referenced this issue Feb 13, 2024
…ator when not loading (brave/brave-ios#7434)

Fix for displaying loading state for asset auto discovery when not discovering assets due to rate limiting
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.