Chore/upgrade plaid link sdk#69971
Conversation
|
|
|
The prettier check is failing on main as well. I'll merge main once resolved :) |
|
@rohit9625 The app crashes on launch for me on the iOS 18.3 simulator. |
This comment was marked as resolved.
This comment was marked as resolved.
|
Sorry, I misread the version. On which simulator device do you test, @akinwale? |
|
I just tested on iOS Screen.Recording.2025-09-07.at.8.51.14.AM.mov |
|
Hi, @akinwale |
|
@rohit9625 I'll try a new clean build today. Please merge the latest main as well. |
|
@rohit9625 I am no longer experiencing the crash on launch after a full rebuild on iOS. I am seeing this error on iOS and Android native the first time I tap on the Add bank account button. Could you check and verify if it was introduced by this PR or if it is happening on main?
|
|
Hi @akinwale, I tested on main and can confirm that the issue is not caused by this PR. This was reproducible even before I created this PR. Screen.Recording.2025-09-09.at.11.13.46.AM.mov |
Reviewer Checklist
Screenshots/VideosAndroid: HybridApp69971-android-hybrid.mp4Android: mWeb Chrome69971-android-chrome.mp4iOS: HybridApp69971-ios-hybrid.mp4iOS: mWeb Safari69971-ios-safari.mp4MacOS: Chrome / Safari69971-web.mp4MacOS: Desktop69971-desktop.mp4 |
|
I'm going to do an adhoc build for this and request QA do testing on plaid flows |
|
Seems like Adhoc is failing, @justinpersaud on main. |
|
🚧 @justinpersaud has triggered a test Expensify/App build. You can view the workflow run here. |
This comment has been minimized.
This comment has been minimized.
|
You want a new adhoc build from this branch? I can do another one if that is what we want. |
|
Yes, please trigger a new build. Thanks 🙂 |
|
🚧 @justinpersaud has triggered a test Expensify/App build. You can view the workflow run here. |
|
Hey @justinpersaud, the build is failing because of Node version mismatch. I think we need to install node version 20.19.4 for Hybrid codebase. Seems like it's recently upgraded. |
|
🧪🧪 Use the links below to test this adhoc build on Android, iOS, Desktop, and Web. Happy testing! 🧪🧪
|
|
🚧 @justinpersaud has triggered a test Expensify/App build. You can view the workflow run here. |
|
🧪🧪 Use the links below to test this adhoc build on Android, iOS, Desktop, and Web. Happy testing! 🧪🧪
|
|
Looking into the node issue |
|
Guys, both Android & iOS adhoc build issues are unrelated to this PR and should already be fixed
I'd recommend merging the latest main I also can't find the cc @rohit9625 |
|
Thanks for the bump, @war-in |
|
I just saw that the checks are failing on main. Should I really merge now or wait for them to resolve on main? |
|
@rohit9625 Let's wait for the fix... |
|
Sure, thanks 😊 |
|
@rohit9625 PR with the fix has been merged 🚀 |
|
@rohit9625 you've got conflicts here |
|
Thanks for the bump, @roryabraham and @war-in The checks were failing when I checked yesterday. But now everything looks great. So, I'm merging main resolving the conflicts. |
|
Hi @justinpersaud, I freshly merged main to this branch. Could you please trigger an Adhoc build as previous ones were failing for Android and iOS? |
|
🚧 @justinpersaud has triggered a test Expensify/App build. You can view the workflow run here. |
|
🧪🧪 Use the links below to test this adhoc build on Android, iOS, Desktop, and Web. Happy testing! 🧪🧪
|
|
Build is done. Can you check if the issue is resolved now? |
|
The above issue has been resolved now :) Screen.Recording.2025-09-23.at.7.48.39.PM.movThank you so much, everyone. Please proceed with final review, @justinpersaud |
|
✋ 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 https://github.com/justinpersaud in version: 9.2.17-0 🚀
|
|
🚀 Deployed to production by https://github.com/roryabraham in version: 9.2.17-6 🚀
|


Explanation of Change
We were using
react-native-plaid-link-sdkversion 11.11.0, which doesn't handle edge-to-edge insets for Android 15+ devices. So, I upgraded the library version to 12.4.0 as it was the latest stable release, which handles edge-to-edge correctly.I also removed an outdated patch for
react-native-plaid-link-sdkas the patch fix was included in version 11.12.1. So, no need to maintain that patch.Fixed Issues
$ #69115
PROPOSAL: $ #69115 (comment)
Tests
Offline tests
Can't test in Offline Mode as the
Add Bank Accountbutton is disabled in Offline Mode which is the expected behavior.QA Steps
Same as tests
// TODO: These must be filled out, or the issue title must include "[No QA]."
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectioncanBeMissingparam foruseOnyxtoggleReportand notonIconClick)src/languages/*files and using the translation methodSTYLE.md) were followedAvatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.ScrollViewcomponent to make it scrollable when more elements are added to the page.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
Android: Native
Screen.Recording.2025-09-05.at.11.10.27.PM.mov
Android: mWeb Chrome
Screen_recording_20250905_232035.mp4
iOS: Native
Screen.Recording.2025-09-05.at.10.47.05.PM.mov
iOS: mWeb Safari
Screen.Recording.2025-09-05.at.10.50.58.PM.mov
MacOS: Chrome / Safari
Screen.Recording.2025-09-05.at.10.53.24.PM.mov
MacOS: Desktop
Screen.Recording.2025-09-05.at.11.24.47.PM.mov