[FCLite] Add FC Lite integration in FC Example app #4694
Merged
+140
−11
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mats-stripe
commented
Mar 20, 2025
| } else { | ||
| assertionFailure("unable to convert `configurationData` to a `configurationString`") | ||
| } | ||
| let configurationString = String(decoding: configurationData, as: UTF8.self) |
There was a problem hiding this comment.
Drive-by fix. Our linter was yelling about this line
mats-stripe
force-pushed
the
mats/fc_lite_interface_and_models
branch
from
caeb5ea to
065992c
Compare
mats-stripe
force-pushed
the
mats/fc_lite_in_fc_example_app
branch
2 times, most recently
from
ac4a6ca to
e122ef5
Compare
tillh-stripe
previously approved these changes
Mar 21, 2025
| @@ -37,11 +37,13 @@ final class PlaygroundConfiguration { | |||
| enum IntegrationType: String, CaseIterable, Identifiable, Hashable { | |||
| case standalone = "standalone" | |||
| case paymentElement = "payment_element" | |||
| case fcLite = "fc_lite" | |||
There was a problem hiding this comment.
Not blocking, but would it make sense to make FC Lite its own setting? This way, you can run both standalone and MPE with both full and lite SDKs.
There was a problem hiding this comment.
Done in 1172475 - added it to the SDK type:
mats-stripe
force-pushed
the
mats/fc_lite_in_fc_example_app
branch
from
e122ef5 to
1172475
Compare
tillh-stripe
approved these changes
Mar 25, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Adds an entry point for FC Lite in our Financial Connections Example app. FC Lite currently isn't implemented, so this doesn't currently do anything, but it will come in handy for my next PRs that build out this integration.
Motivation
Will allow testing FC Lite once it's implemented.
Testing
Changelog
N/a