Closed marcaaron closed 3 years ago
main
locallystaging
locally to see if this can be reproduced there@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.
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.
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.
Oh that and I think the API changed because everything breaks once you pick an account. Nice! React Native ftw π
write once, break everywhere
:trollface:
Edit: JK it's fine my phone got unplugged from metro π
Seeing an error that looks like this now:
ERROR Invariant Violation: No callback found with cbID 7424 and callID 3712 for PlaidAndroid.startLinkActivityForResult - most likely the callback was already invoked. Args: '[{"publicToken":"sadfasdfasdf","metadata":{"metadataJson":"{\"institution\":{\"name\":\"Chase\",\"institution_id\":\"ins_3\"},\"account\":{\"id\":null,\"name\":null,\"type\":nul...(truncated)...","linkSessionId":"43241cdf-5ac7-465c-ae61-cbfcb2e0ad29","institution":{"name":"Chase","id":"ins_3"},"accounts":[{"subtype":"checking","mask":"0000","name":"Plaid Checking","type":"depository","id":"xaaKnjeRvRfjdD1evna4fwLV8mDQWQU9kZ8Ga"},{"subtype":"savings","mask":"1111","name":"Plaid Saving","type":"depository","id":"dooZ6LnN1Nu8gB3rwNXzSLWnNyPawaiPlq7Wj"}]}}]', js engine: hermes
Ok I think maybe it was just a caching issue. Seems to be working now on Android. Moving on to iOS testing.
Thanks for looking into this. Let us know when the PR comes off hold and can be tested.
Leaving a reminder here that this PR should be retested after this is merged.
Hey @Julesssss I re-tested your branch and looks good with the changes. No password field appears and able to advance further without one.
Perfect, thanks again for looking into this and for following up.
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 π¬
:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.
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.
If you havenβt already, check out our contributing guidelines for onboarding and email contributors@expensify.com 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:
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