Plaid modal is no longer opening on Android

[marcaaron]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

Action Performed:

Try to add a bank account via Plaid

Expected Result:

Plaid modal opens and we are able to do so (works on staging)

Actual Result:

White screen + nothing happens

Workaround:

No

Platform:

• Android

Version Number:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:

View all open jobs on GitHub

[marcaaron]

Coming in from #4944 (comment)

[marcaaron]

1. The issue was reproduced on main locally
2. The issue is not reproduced on the latest Play Store build
3. Going to try to test staging locally to see if this can be reproduced there

[marcaaron]

@Julesssss @luacmartins the issue is reproducible on the staging branch too. Which likely means this isn't a deploy blocker. But should be resolved so we can test this on dev.

I'll try to see what else could be causing this. It was working... and now isn't... but the code changes around this stuff have been pretty minimal so I'm guessing some kind of dependency broke this. But honestly not too sure.

[marcaaron]

Ok so I updated the react-native-plaid-link-sdk package to see if that would help at all and it did. I think updating to the latest version of React Native must have broken this. I'm going to push up the version bump and test to make sure that iOS is still good as well.

[marcaaron]

The only thing I'm noticing is that the Plaid modal is well... super slow to load... but it does work eventually.
Maybe we can create another issue to address that or bring it to Plaid's attention.

[marcaaron]

Ok I think maybe it was just a caching issue. Seems to be working now on Android. Moving on to iOS testing.

[Julesssss]

Thanks for looking into this. Let us know when the PR comes off hold and can be tested.

[Julesssss]

Leaving a reminder here that this PR should be retested after this is merged.

[marcaaron]

Hey @Julesssss I re-tested your branch and looks good with the changes. No password field appears and able to advance further without one.

Edited_20210901_115820.mp4

[Julesssss]

Perfect, thanks again for looking into this and for following up.

[Julesssss]

That reminder was for myself, but I see it wasn't that clear now, I hope you didn't feel like I was asking you to do that 😬

[botify]

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

[marcaaron]

That reminder was for myself, but I see it wasn't that clear now, I hope you didn't feel like I was asking you to do that 😬

Ah nah yer good ! Just trying to do ya a solid.

[botify]

🚀 Deployed to staging by @luacmartins in version: 1.0.92-1 🚀

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

[botify]

🚀 Deployed to production by @roryabraham in version: 1.0.93-1 🚀

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