Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sync spinner missing ~70% of the time I foreground the app #1576

Closed
ara4n opened this issue Aug 28, 2023 · 2 comments
Closed

Sync spinner missing ~70% of the time I foreground the app #1576

ara4n opened this issue Aug 28, 2023 · 2 comments
Labels
A-Sliding-Sync O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@ara4n
Copy link
Member

ara4n commented Aug 28, 2023

Steps to reproduce

  1. Have a big account
  2. Foreground the app
  3. See no sync spinner, but rooms are ~5 minutes old
  4. After waiting for up to ~10s (Launch time has regressed substantially. #1509) the room list suddenly updates.

This began after #1562 was merged

@stefanceriu's theory is that the app thinks a background refresh is in progress, hence not showing a spinner. But empirically it isn't.

Outcome

What did you expect?

Spinner when the app is showing me stale data and trying to sync.

What happened instead?

No spinner

Your phone model

No response

Operating system version

No response

Application version

364

Homeserver

No response

Will you send logs?

Yes

@ara4n ara4n added the T-Defect label Aug 28, 2023
@ara4n
Copy link
Member Author

ara4n commented Aug 28, 2023

having disabled background refresh in OS settings, i'm not seeing this misbehaviour any more.

@stefanceriu: it sounds like background refresh could very legitimately get stuck, given it might have a lot of data to sync in the bg, be very ratelimited in the app, and also be on terrible data (given the user could be wandering around making no attempt for the app to be in good connectivity given they're not actively using it - e.g. on a train). Is there a way for the app to knife bg refresh and start over it's stuck when it's foregrounded, in order to get the sync spinner back again?

@Velin92 Velin92 added S-Minor Impairs non-critical functionality or suitable workarounds exist O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience A-Sliding-Sync labels Sep 4, 2023
@manuroe
Copy link
Member

manuroe commented Jan 19, 2024

Seems to be solved now

@manuroe manuroe closed this as completed Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Sliding-Sync O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

No branches or pull requests

3 participants