Merge pull request #1569 from stripe-ios/kg-smallcleanup

Financial Connections: small native clean-up/changes
stripe · Nov 7, 2022 · fd1f166 · fd1f166
2 parents 63844ce + f59586b
commit fd1f166
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 6 deletions.
diff --git a/...cialConnections/Source/API Bindings/Models/FinancialConnectionsAuthorizationSession.swift b/...cialConnections/Source/API Bindings/Models/FinancialConnectionsAuthorizationSession.swift
@@ -102,9 +102,11 @@ enum FinancialConnectionsPartner {
         case .mx:
             return Image.mx.makeImage()
         case .wellsFargo:
-            return nil // TODO(kgaidis): do we need a wells fargo icon?
+            // we never show icons for direct integrations
+            return nil
         case .trueLayer:
-            return nil // TODO(kgaidis): do we need a true layer icon?
+            // icon not needed until EU support
+            return nil
         }
     }
 }
diff --git a/...nancialConnections/StripeFinancialConnections/Source/Native/Consent/ConsentBodyView.swift b/...nancialConnections/StripeFinancialConnections/Source/Native/Consent/ConsentBodyView.swift
@@ -59,9 +59,9 @@ private func CreateLabelView(
     ])
 
     let label = ClickableLabel(
-        font: UIFont.stripeFont(forTextStyle: .detail),
-        boldFont: UIFont.stripeFont(forTextStyle: .detailEmphasized),
-        linkFont: UIFont.stripeFont(forTextStyle: .detailEmphasized),
+        font: UIFont.stripeFont(forTextStyle: .body),
+        boldFont: UIFont.stripeFont(forTextStyle: .bodyEmphasized),
+        linkFont: UIFont.stripeFont(forTextStyle: .bodyEmphasized),
         textColor: .textSecondary
     )
     label.setText(text, action: action)

diff --git a/...nancialConnections/StripeFinancialConnections/Source/Native/PartnerAuth/PrepaneView.swift b/...nancialConnections/StripeFinancialConnections/Source/Native/PartnerAuth/PrepaneView.swift
@@ -34,7 +34,6 @@ final class PrepaneView: UIView {
                 return institutionIconView
             }()),
             title: String(format: STPLocalizedString("Link with %@", "The title of the screen that appears before a user links their bank account. The %@ will be replaced by the banks name to form a sentence like 'Link with Bank of America'."), institutionName),
-            // TODO(kgaidis): do we need a "we will only share the requested data" subtitle addition?
             subtitle: String(format: STPLocalizedString("A new window will open for you to log in and select the %@ account(s) you want to link.", "The description of the screen that appears before a user links their bank account. The %@ will be replaced by the banks name, ex. 'Bank of America'. "), institutionName),
             contentView: {
                 let clearView = UIView()