-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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 blank screen on IOU confirm page #2976
Conversation
Unfortunately I've got to run and didn't have time to finish testing this on all platforms. Will resume in the AM |
@stitesExpensify This passed testing on all platforms and is ready for your review :) |
Android - Chat - Nothing happened after clicked Request Money buttonExpected ResultThe screen should redirect to LHN after clicked Request Money button the Actual ResultNothing happened after clicked Request Money button Action Performed
PlatformAndroid ✔️ Notes/Images/Video Bug5074926_Screen_Recording_20210518-170423_Expensifycash__1_.mp4 |
@isagoico This goes a step further than the QA steps for this PR. Could you please create a separate issue? |
Sure! Issue here #2995 |
Details
The
option.participantsList
may be undefined (per it's prop types here). So this PR ensures that we're safely checking the length of that array.Fixed Issues
Fixes #2969
Tests / QA Steps
+
iconNext
Tested On
Screenshots
Web
Mobile Web
Desktop
iOS
Android