Skip to content

Conversation

@Twigz
Copy link
Collaborator

@Twigz Twigz commented Oct 22, 2025

Summary

Updated iOS/Android SDKs, mainly to support the new Identity changes made by the Identity team.

This update in particular handles icon display during Identity Verification, as well as auto-dismissal of verification completion.

Motivation

Changes to the Identity framework were necessary to support some Crypto Onramp features.

Testing

  • I tested this manually
  • I added automated tests

Documentation

Select one:

  • I have added relevant documentation for my changes.
  • This PR does not result in any developer-facing changes.

Screenshots

Android iOS

@Twigz Twigz requested review from a team and charliecruzan-stripe as code owners October 22, 2025 18:09
nextActionMap.putString("qrCodeUrl", it.qrCodeUrl)
}
}
NextActionType.DisplayPromptPayDetails -> {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Curious why iOS doesn't need a similar change?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That, is a good question. The answer appears to be, it probably should, but hasn't kept up to date for a while. The reason it's not flagged is because it uses a default case in Swift.

default: // .useStripeSDK, .BLIKAuthorize, .unknown
    return nil
}

Currently the only implemented cases are:

verifyWithMicrodeposits
redirectToURL
weChatPayRedirectToApp
alipayHandleRedirect
OXXODisplayDetails
boletoDisplayDetails
konbiniDisplayDetails

Based on comparing the available iOS cases to implemented Android cases, likely the following need to be added for functionality

    case payNowDisplayQrCode
    case promptpayDisplayQrCode
    case swishHandleRedirect
    case multibancoDisplayDetails

with a few others being defined but being a no-op (upiAwaitNotification and cashAppRedirectToApp)

I can handle that as part of this PR if what I've explained is correct that this is an oversight due to a default switch case. Let me know @lng-stripe / @mats-stripe.

@Twigz Twigz closed this Nov 6, 2025
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.

3 participants