fix(billing): Avoid double OTP during add card via stripe#6371
Conversation
There was a problem hiding this comment.
Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.
Codecov Report❌ Patch coverage is
❌ Your patch check has failed because the patch coverage (71.23%) is below the target coverage (75.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## develop #6371 +/- ##
===========================================
- Coverage 49.85% 49.83% -0.02%
===========================================
Files 944 945 +1
Lines 78378 78394 +16
Branches 350 355 +5
===========================================
- Hits 39073 39068 -5
- Misses 39281 39303 +22
+ Partials 24 23 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Codecov Report❌ Patch coverage is
❌ Your patch status has failed because the patch coverage (71.23%) is below the target coverage (75.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## develop #6371 +/- ##
===========================================
- Coverage 49.85% 49.83% -0.02%
===========================================
Files 944 945 +1
Lines 78378 78394 +16
Branches 350 355 +5
===========================================
- Hits 39073 39068 -5
- Misses 39281 39303 +22
+ Partials 24 23 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Removed setup intent and going only with payment intent and storing the card information as part of it. Verfied in local and this persists the card info for future usage