stripe-android icon indicating copy to clipboard operation
stripe-android copied to clipboard

Use the generic title "Front of identity document" instead of "Front of identity card"

Open googol7 opened this issue 1 year ago • 1 comments
trafficstars

See #8511

This change is similar to:

  • https://github.com/stripe/stripe-ios/pull/3632/files#diff-2a6b2ccc2c3c46f603c78f8eba3e6f982503c9f60c1731ea09926a22671beff3R17-R23

Summary

Motivation

Because currently the title is always "Front of identity card" regardless of what is configured in options.document.allowed_types, users upload their "Identity card" (German: "Personalausweis") instead of the "Driver's license" (German: Führerschein"). So the more generic term "Identity document" should be used. This approach was also changed recently in stripe-ios here.

The optimal solution would be that the title corresponds exactly with options.document.allowed_types. This is how the web version already does it.

Example:

options.document.allowed_types string used
["driving_license"] <string name="stripe_front_of_dl">Front of driver\'s license</string>
["id_card"] <string name="stripe_front_of_id">Front of identity card</string>
["passport"] <string name="stripe_passport">Passport</string>
["driving_license", "id_card", "passport"] <string name="stripe_front_of_id_document">Front of identity document</string>

I've seen that with this commit in the PR

  • https://github.com/stripe/stripe-android/pull/8671

these new translations were added:

<string name="stripe_front_of_id_document">Front of identity document</string>
<string name="stripe_back_of_id_document">Back of identity document</string>

but they are not used in the code yet.

Testing

  • [ ] Added tests
  • [x] Modified tests
  • [ ] Manually verified

Screenshots

Before After
before screenshot after screenshot

Changelog

googol7 avatar Sep 04 '24 19:09 googol7

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Sep 04 '24 19:09 CLAassistant

@aywang-stripe @awush-stripe We have verification sessions every day, where users upload the front side of their driver's license, but upload the backside of their ID card (German: "Personalausweis") because these titles are wrong. We need the backside of the driver's license.

The Stripe support wrote:

I wanted to provide you with some clarification regarding our ID verification process. We still consider the front side of the document if it is recognized as a driver's license, especially if we are unable to extract clear information from the back of these IDs. In such cases, we will trust the data provided on the front of the IDs.

Additionally, the Stripe Identity team has informed us that there are currently no immediate plans to work on a feature that would allow us to also consider the back of the IDs during the verification process.

We apologize for the inconvenience this may have caused. If you have any further questions or concerns, please feel free to reach out.

googol7 avatar Oct 28 '24 08:10 googol7

@aywang-stripe @awush-stripe Thanks for approving this PR. Does it need some more approval? When do you expect it to be merged? Thanks!

googol7 avatar Nov 14 '24 13:11 googol7

Closing in favor of https://github.com/stripe/stripe-android/pull/9829 (fixes error relating to reference to stripe_front_of_id_document_selected message that does not exist)

kentwilliams-stripe avatar Dec 25 '24 00:12 kentwilliams-stripe