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

Stale UI state on second login #17

Closed
richiemccoll opened this issue Feb 17, 2021 · 11 comments
Closed

Stale UI state on second login #17

richiemccoll opened this issue Feb 17, 2021 · 11 comments
Assignees
Labels
bug Something isn't working

Comments

@richiemccoll
Copy link
Contributor

richiemccoll commented Feb 17, 2021

UPDATE: the PR which fixes this issue has been merged but it needs an expo SDK release to be available. See expo/expo#11503

I noticed some stale UI state after logging in with Google. This only happens after I log in and logout once. I was able to reproduce it on both iPhone and Android.

RPReplay_Final1613578363.MP4

Steps to Reproduce

  • Login with Google & logout
  • Login again with the same account

Expected

  • User arrives on the Home screen

Actual

  • User is stuck on the Auth screen
@richiemccoll richiemccoll added the bug Something isn't working label Feb 17, 2021
@simoneb
Copy link
Member

simoneb commented Feb 17, 2021

it works for me

@richiemccoll
Copy link
Contributor Author

@simoneb I've updated the PR description with a screen recording to illustrate the bug.

@simoneb
Copy link
Member

simoneb commented Feb 17, 2021

I tried the steps but could not reproduce it. Let's sync on this tomorrow.

@n370
Copy link
Contributor

n370 commented Mar 2, 2021

Seem like we have company here. I've tried the changes seen at expo/expo#11714 (comment) and was able to get a consistent behavior where the login would work every other way.

There is a change coming that may get this one fixed for us: expo/expo#11503 (review).

@simoneb
Copy link
Member

simoneb commented Mar 2, 2021

Good catch!

@tiagoheliob
Copy link
Contributor

Issued was fixed on this PR expo/expo#11503

@simoneb
Copy link
Member

simoneb commented Sep 29, 2021

@tiagoheliob the issue was fixed in expo but did you check if we're using the expo version which includes that fix? Because I don't think we updated expo since a long time, hence the fix is not in the version we're using here. I'm reopening the issue, if you want to close it can you please double check this?

@simoneb simoneb reopened this Sep 29, 2021
@simoneb
Copy link
Member

simoneb commented Sep 29, 2021

I've also created #153 to upgrade expo to latest

@simoneb
Copy link
Member

simoneb commented Oct 1, 2021

We need to verify whether this is still an issue after the application underwent further changes and after the expo upgrade, which may have got us the fix.

@salmanm
Copy link
Contributor

salmanm commented Oct 22, 2021

Looks like this is fixed now. I tried logging in/out and all seemed fine on my phone

@salmanm salmanm closed this as completed Oct 22, 2021
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 a pull request may close this issue.

5 participants