Skip to content
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

Identity] Support the 'closed' field in submit endpoint #2700

Merged

Conversation

ccen-stripe
Copy link
Contributor

Summary

Introduced a new closed boolean in submit API endpoint, after submitting

  • When submitted and closed are true -> navigate to success page
  • When submitted=true and closed=false, need to fallback, check missings and fallback accordingly - currently this only happens when phone type verification fallback to document

Android change: stripe/stripe-android#6958

Motivation

Support phoneV

Testing

  • Added tests
  • Modified tests
  • Manually verified

Changelog

@ccen-stripe ccen-stripe requested review from a team as code owners July 1, 2023 01:02
@ccen-stripe ccen-stripe requested review from porter-stripe and removed request for a team July 1, 2023 01:02
@ccen-stripe ccen-stripe force-pushed the ccen/supportCannotVerify branch from f7fdc58 to 3cdcaf0 Compare July 17, 2023 17:41
@ccen-stripe ccen-stripe force-pushed the ccen/IDPROD-5537/phoneVerificationBase branch from ce2e43e to 3ca5e7f Compare July 17, 2023 17:46
@ccen-stripe ccen-stripe force-pushed the ccen/supportCannotVerify branch from 753daec to 8972a51 Compare July 17, 2023 17:47
@ccen-stripe ccen-stripe requested review from vardges-stripe and removed request for porter-stripe July 17, 2023 22:23
Copy link
Contributor

@vardges-stripe vardges-stripe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just the viewwillappear question

Copy link
Contributor

@vardges-stripe vardges-stripe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ty

@ccen-stripe
Copy link
Contributor Author

Have to move updateUI to viewWillAppear to make sure the UI is not incorrectly picking up the previous VC's content

Before After
before after

@ccen-stripe ccen-stripe merged commit ee150fd into ccen/IDPROD-5537/phoneVerificationBase Jul 18, 2023
@ccen-stripe ccen-stripe deleted the ccen/supportCannotVerify branch July 18, 2023 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants