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

Add support for id_consent_content pane #4644

Draft
wants to merge 4 commits into
base: mats/switch_to_institution_selected_api
Choose a base branch
from

Conversation

mats-stripe
Copy link
Collaborator

Summary

As part of the institution picker first experiment, a new pane id_consent_content is added. It's mostly identical to the consent pane, with a few minor adjustments.

Motivation

Support institution picker first experiment.

Testing

Coming soon

Changelog

N/a

Copy link

github-actions bot commented Mar 10, 2025

🚨 New dead code detected in this PR:

FinancialConnectionsGenericInfoScreen.swift:65 warning: Struct 'TypeDecodingContainer' is unused
IDConsentContentViewController.swift:20 warning: Parameter 'viewController' is unused

Please remove the dead code before merging.

If this is intentional, you can bypass this check by adding the label skip dead code check to this PR.

ℹ️ If this comment appears to be left in error, double check that the flagged code is actually used and/or make sure your branch is up-to-date with master.

@mats-stripe mats-stripe force-pushed the mats/add_id_consent_content_pane branch from 9b03a9a to 72822f5 Compare March 10, 2025 18:53

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@@ -106,29 +106,6 @@ final class ConsentLogoView: UIView {
}
}

private func CreateRoundedLogoView(urlString: String) -> UIView {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Moved to be shared with the new pane


import Foundation

struct FinancialConnectionsIDConsentContent: Decodable {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@mats-stripe mats-stripe force-pushed the mats/add_id_consent_content_pane branch from 499e215 to 412209b Compare March 14, 2025 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant