Skip to content

Conversation

KunJeongPark
Copy link
Collaborator

@KunJeongPark KunJeongPark commented Sep 16, 2025

Reason for changes

  • Allow app switch options in demo app with supporting data structures

Summary of changes

  • App switch toggle
  • Make views on vaulting take in UI selection options, avoid business logic
  • app_switch_context created in experience_context if app switch selected.
  • only send in app switch URL to AppSwithContext initializer

Checklist

  • Added a changelog entry

Authors

List GitHub usernames for everyone who contributed to this pull request.

@KunJeongPark KunJeongPark changed the title demo app changes for vault app switch Demo App changes for Vault App Switch Sep 16, 2025
init(appSwitchContext: AppSwitchContext? = nil) {
self.appSwitchContext = appSwitchContext
if appSwitchContext != nil {
self.returnUrl = baseReturnHost + "/success"
Copy link
Collaborator Author

@KunJeongPark KunJeongPark Sep 17, 2025

Choose a reason for hiding this comment

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

I am checking with checkout team on this, whether this can remain custom url scheme.
Ideally, we have ability to have happy path return_url back from PayPal app from the app switch context app_url.
I will make changes on it on subsequent PRs depending on answer we get.
For web checkout, iOS SDK constructs the ASWebAuthentication launch url with return_uri param that contains SDK specific custom url scheme, "sdk.ios.paypal"
So we can default to this in case of ineligibility or app not installed.

@KunJeongPark KunJeongPark marked this pull request as ready for review September 17, 2025 15:52
Copy link
Collaborator

@sshropshire sshropshire left a comment

Choose a reason for hiding this comment

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

🌊

@KunJeongPark KunJeongPark merged commit 4c59ac9 into feature/app-switch Sep 21, 2025
4 checks passed
@KunJeongPark KunJeongPark deleted the more-demo-app-changes branch September 21, 2025 06:17
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