-
Notifications
You must be signed in to change notification settings - Fork 984
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
[#7848] Fix crash on logout with opened transactions details #8101
Conversation
Pull Request Checklist
|
Jenkins Builds
|
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.
why do we start having such issues?
@flexsurfer because screens are still rendered when we switch to another tab. And if screen gets broken state before it is unmounted we get such exceptions. |
98% of end-end tests have passed
Failed tests (1)Click to expand
Passed tests (47)Click to expand
|
Tested on IOS 11.4.1 (IPhone 7), Android 8 (LG V20):
E2E failure is not related to this PR. |
7293c99
to
62e94ed
Compare
✔️ status-react/prs/ios/PR-8101#2 🔹 ~22 min 🔹 62e94ed 🔹 📦 ios package |
fix #7848
status: ready