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

Clear storage immediately on sign out #4614

Merged

Conversation

kidroca
Copy link
Contributor

@kidroca kidroca commented Aug 12, 2021

cc @iwiznia @francoisl

Details

Waiting for set to complete gives enough time for other connections (triggered by authToken change) to be initiated with cached data
This causes a bug where the SignIn component does not have correct state
More details are available in the linked issue

Fixed Issues

$ #4595

Tests

  1. Launch the mobile app
  2. Switch a few chats
  3. Log out

You should be on the initial Login screen, where you need to input your email

QA Steps

Same as above

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

New.Expensify.-.Google.Chrome.2021-08-12.21-49-38.mp4

Mobile Web

Android.Emulator.-.Pixel_2_API_29_5554.2021-08-12.21-51-05.mp4

Desktop

image

iOS

Screen.Recording.2021-08-12.at.21.57.39.mov

Android

Android.Emulator.-.Pixel_2_API_29_5554.2021-08-12.21-07-17.mp4

Waiting for `set` to complete gives enough time for other connections (triggered
by the authToken change) to be initiated with cached data
This causes a bug where the SignIn component does not have correct state
@kidroca kidroca requested a review from a team as a code owner August 12, 2021 18:39
@MelvinBot MelvinBot requested review from iwiznia and removed request for a team August 12, 2021 18:39
@kidroca
Copy link
Contributor Author

kidroca commented Aug 12, 2021

I'm testing the rest of the platforms (iOS still building...)

@github-actions
Copy link
Contributor

⚠️ ⚠️ Heads up! This pull request has the CP Staging label. ⚠️ ⚠️
Merging it will cause it to be immediately deployed to staging, even if the open StagingDeployCash deploy checklist is locked.

Copy link
Contributor

@iwiznia iwiznia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. @francoisl please review and merge once testing has been done in all platforms. Then please let QA know they can retest in the issue 🙏

@kidroca
Copy link
Contributor Author

kidroca commented Aug 12, 2021

Tested on all platform and added videos in the description.
This is ready for review.

@francoisl francoisl merged commit 3f72e03 into Expensify:main Aug 12, 2021
github-actions bot pushed a commit that referenced this pull request Aug 12, 2021
…storage-fix

Clear storage immediately on sign out

(cherry picked from commit 3f72e03)
@OSBotify
Copy link
Contributor

🚀 Cherry-picked to staging by @francoisl in version: 1.0.85-2 🚀

platform result
🤖 android 🤖 failure ❌
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@botify
Copy link

botify commented Aug 17, 2021

This has been deployed to production and is now subject to a 7-day regression period.
If no regressions arise, payment will be issued on 2021-08-24. 🎊

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @isagoico in version: 1.0.85-9 🚀

platform result
🤖 android 🤖 failure ❌
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@OSBotify
Copy link
Contributor

🚀 Deployed to staging by @francoisl in version: 1.0.85-10 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@botify
Copy link

botify commented Aug 25, 2021

This has been deployed to production and is now subject to a 7-day regression period.
If no regressions arise, payment will be issued on 2021-09-01. 🎊

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @roryabraham in version: 1.0.86-11 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 failure ❌
🕸 web 🕸 success ✅

@botify
Copy link

botify commented Aug 30, 2021

This has been deployed to production and is now subject to a 7-day regression period.
If no regressions arise, payment will be issued on 2021-09-06. 🎊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants