stripe-android
stripe-android copied to clipboard
Use the generic title "Front of identity document" instead of "Front of identity card"
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
@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.
@aywang-stripe @awush-stripe Thanks for approving this PR. Does it need some more approval? When do you expect it to be merged? Thanks!
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)