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

fix: prevent incorrect offline->online transitions without successful… #617

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

irishrain
Copy link
Contributor

While testing #610 I found a race condition that sometimes logs the user out when coming out of offline mode when the network is bad. #610 and this should fix #414 for good.

@irishrain irishrain force-pushed the fix/offline-mode-race-condition branch from 4ab9571 to d9f3402 Compare December 31, 2024 20:59
@TomBursch TomBursch added the bug Something isn't working label Jan 3, 2025
@TomBursch
Copy link
Owner

Is this superseded now by #610? I noticed that you fixed that there too in a slightly different way.

@irishrain
Copy link
Contributor Author

irishrain commented Jan 3, 2025

Oh no, I must have somehow messed up with git branches.
This is still needed, the change that snuck into #610 only avoids the logout, but then the app will be stuck in offline mode.

@irishrain irishrain force-pushed the fix/offline-mode-race-condition branch from d9f3402 to 83b10ac Compare January 4, 2025 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Logged out after coming back from offline mode
2 participants