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

Extract HostedAuthUrlBuilder to StripeCore #4733

Merged
merged 2 commits into from
Mar 28, 2025

Conversation

mats-stripe
Copy link
Collaborator

@mats-stripe mats-stripe commented Mar 27, 2025

Summary

Extracts the hosted auth URL building logic from the FC SDK into StripeCore for reuse in FC Lite. This will ensure we pass along the same parameters to the hosted auth flow, and will fix bugs we found during the bug bash, like fixing Panther!

There should be no functional changes introduced for the full SDK here.

Motivation

Ensures consistency in the hosted auth flow URLs we launch from either FC Lite or the full FC SDK.

Testing

Lots of unit tests added!

Changelog

N/a - FC Lite isn't released yet.

tillh-stripe
tillh-stripe previously approved these changes Mar 27, 2025
@mats-stripe mats-stripe force-pushed the mats/extract_hostedauthurl_builder_to_stripecore branch from 2bb2d4d to e975ba3 Compare March 27, 2025 17:28
@mats-stripe mats-stripe marked this pull request as ready for review March 27, 2025 17:28
@mats-stripe mats-stripe requested review from a team as code owners March 27, 2025 17:28
@mats-stripe mats-stripe force-pushed the mats/fc_lite_availability_in_mpe branch from ee0a191 to f793248 Compare March 27, 2025 19:27
@mats-stripe mats-stripe force-pushed the mats/extract_hostedauthurl_builder_to_stripecore branch from e975ba3 to 10dd6ee Compare March 27, 2025 19:29
@mats-stripe mats-stripe force-pushed the mats/fc_lite_availability_in_mpe branch from 58d8ed9 to 14d244d Compare March 28, 2025 15:49
@mats-stripe mats-stripe force-pushed the mats/extract_hostedauthurl_builder_to_stripecore branch from 10dd6ee to d3efb4e Compare March 28, 2025 15:50
Base automatically changed from mats/fc_lite_availability_in_mpe to master March 28, 2025 17:01
@mats-stripe mats-stripe dismissed tillh-stripe’s stale review March 28, 2025 17:01

The base branch was changed.

@mats-stripe mats-stripe force-pushed the mats/extract_hostedauthurl_builder_to_stripecore branch from d3efb4e to 61f6ce0 Compare March 28, 2025 19:28
@mats-stripe mats-stripe merged commit 845fcbd into master Mar 28, 2025
6 checks passed
@mats-stripe mats-stripe deleted the mats/extract_hostedauthurl_builder_to_stripecore branch March 28, 2025 21:14
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