-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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: lag between language switch on login page #5300
Conversation
src/libs/actions/Session.js
Outdated
@@ -298,6 +298,16 @@ function continueSessionFromECom(accountID, validateCode, twoFactorAuthCode) { | |||
}); | |||
} | |||
|
|||
/** | |||
* Clear the creadentials and partial sign in session so the user can taken back to first Login step |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor typo: creadentials
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests well, but lets fix the typo.
Oops. Updated @Julesssss |
Holding due to N6. |
@parasharrajat this can be merged now, you just need to resolve the conflict 👍 |
e41e7e2
to
7e1221e
Compare
@Julesssss All set. |
Ready to merge. |
@parasharrajat thanks, the iOS build takes forever 😴 |
We've tested this against the latest |
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
🚀 Deployed to staging by @Julesssss in version: 1.1.8-10 🚀
|
🚀 Deployed to production by @roryabraham in version: 1.1.10-2 🚀
|
Details
Fixed Issues
$ #5068 (comment)
Tests | QA Steps
Tested On
Screenshots
Web
o-s-w.mp4
Mobile Web
o-s-m.mp4
Desktop
iOS
Android
o-s-a.mp4