-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[No QA] Add the enableCardAfterVerified param to additionalData #3741
Conversation
Off hold now ! |
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.
Nice! Seems related to when we added validation, probably we just need to catch and handle this error. Will create a follow up thanks! |
✋ 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 in version: 1.0.73-4🚀
|
🚀 Deployed to production in version: 1.0.74-0🚀
|
Details
cc @ctkochan22
This should HOLD on https://github.com/Expensify/Auth/pull/5709 to merge + deploy
Fixed Issues
https://github.com/Expensify/Expensify/issues/168429
Tests
additionalData
JSON and verify theenableCardAfterVerified
param is presentQA Steps
❌
Tested On
Screenshots
Web
Mobile Web
Desktop
iOS
Android