fix(GiniHealthSDK): Fix RC bugs reported

GiniComponents/GiniInternalPaymentSDK/Sources/GiniInternalPaymentSDK/PaymentInfo/PaymentInfoConfiguration.swift

@@ -66,7 +66,7 @@ public struct PaymentInfoConfiguration {
 }
 
 public struct PaymentInfoStrings {
-    let giniWebsiteText : String
+    let giniWebsiteText: String
     let giniURLText: String
 
     let questionsTitleText: String

GiniComponents/GiniInternalPaymentSDK/Sources/GiniInternalPaymentSDK/PaymentInfo/PaymentInfoViewModel.swift

@@ -76,11 +76,13 @@ public final class PaymentInfoViewModel {
         let attributedString = attributedString
         let giniRange = (attributedString.string as NSString).range(of: strings.giniWebsiteText)
         attributedString.addLinkToRange(link: strings.giniURLText,
+                                        color: configuration.linksColor,
                                         range: giniRange,
                                         linkFont: linkFont,
                                         textToRemove: Constants.linkTextToRemove)
         let privacyPolicyRange = (attributedString.string as NSString).range(of: strings.answerPrivacyPolicyText)
         attributedString.addLinkToRange(link: strings.privacyPolicyURLText,
+                                        color: configuration.linksColor,
                                         range: privacyPolicyRange,
                                         linkFont: linkFont,
                                         textToRemove: Constants.linkTextToRemove)

GiniComponents/GiniInternalPaymentSDK/Sources/GiniInternalPaymentSDK/PaymentReview/PaymentReviewModel.swift

@@ -54,6 +54,7 @@ public protocol PaymentReviewActionProtocol {
     func updatedPaymentProvider(_ paymentProvider: PaymentProvider)
     func openMoreInformationViewController()
     func presentShareInvoiceBottomSheet(paymentRequestId: String, paymentInfo: PaymentInfo)
+    func paymentReviewClosed(with previousPresentedView: PaymentComponentScreenType?)
 }
 
 /**
@@ -123,6 +124,7 @@ public class PaymentReviewModel: NSObject {
     let bottomSheetConfiguration: BottomSheetConfiguration
     let showPaymentReviewCloseButton: Bool
     var displayMode: DisplayMode
+    var previousPaymentComponentScreenType: PaymentComponentScreenType?
 
     public init(delegate: PaymentReviewProtocol,
                 bottomSheetsProvider: BottomSheetsProviderProtocol,
@@ -142,7 +144,8 @@ public class PaymentReviewModel: NSObject {
                 poweredByGiniConfiguration: PoweredByGiniConfiguration,
                 poweredByGiniStrings: PoweredByGiniStrings,
                 bottomSheetConfiguration: BottomSheetConfiguration,
-                showPaymentReviewCloseButton: Bool) {
+                showPaymentReviewCloseButton: Bool,
+                previousPaymentComponentScreenType: PaymentComponentScreenType?) {
         self.delegate = delegate
         self.bottomSheetsProvider = bottomSheetsProvider
         self.configuration = configuration
@@ -164,6 +167,11 @@ public class PaymentReviewModel: NSObject {
         self.selectionStyleInputFieldConfiguration = selectionStyleInputFieldConfiguration
         self.bottomSheetConfiguration = bottomSheetConfiguration
         self.displayMode = document != nil ? .documentCollection : .bottomSheet
+        self.previousPaymentComponentScreenType = previousPaymentComponentScreenType
+    }
+
+    func viewDidDisappear() {
+        delegate?.paymentReviewClosed(with: previousPaymentComponentScreenType)
     }
 
     func getCellViewModel(at indexPath: IndexPath) -> PageCollectionCellViewModel {
@@ -313,6 +321,7 @@ extension PaymentReviewModel: BanksSelectionProtocol {
      This function notifies the delegate to open the "More Information" view controller.
      */
     public func didTapOnMoreInformation() {
+        previousPaymentComponentScreenType = .bankPicker
         delegate?.openMoreInformationViewController()
     }
 

GiniComponents/GiniInternalPaymentSDK/Sources/GiniInternalPaymentSDK/PaymentReview/PaymentReviewViewController.swift

@@ -21,7 +21,6 @@ public final class PaymentReviewViewController: BottomSheetViewController, UIGes
     private lazy var closeButton = buildCloseButton()
     private lazy var infoBar = buildInfoBar()
     private lazy var infoBarLabel = buildInfoBarLabel()
-    private lazy var containerCollectionView = buildContainerCollectionView()
     private var isInfoBarHidden = true
     lazy var paymentInfoContainerView = buildPaymentInfoContainerView()
     lazy var collectionView = buildCollectionView()
@@ -62,6 +61,10 @@ public final class PaymentReviewViewController: BottomSheetViewController, UIGes
             showInfoBar()
             showInfoBarOnce = false
         }
+        if model.previousPaymentComponentScreenType == .bankPicker {
+            model.openBankSelectionBottomSheet()
+            model.previousPaymentComponentScreenType = nil
+        }
     }
 
     fileprivate func setupViewModel() {
@@ -131,7 +134,9 @@ public final class PaymentReviewViewController: BottomSheetViewController, UIGes
     }
 
     override public func viewDidDisappear(_ animated: Bool) {
+        super.viewDidDisappear(animated)
         unsubscribeFromNotifications()
+        model.viewDidDisappear()
     }
 
     public override var preferredStatusBarStyle: UIStatusBarStyle {
@@ -354,14 +359,6 @@ fileprivate extension PaymentReviewViewController {
 
 //MARK: - Collection View Container
 fileprivate extension PaymentReviewViewController {
-    func buildContainerCollectionView() -> UIStackView {
-        let container = UIStackView(arrangedSubviews: [collectionView, pageControl])
-        container.spacing = 0
-        container.axis = .vertical
-        container.distribution = .fill
-        return container
-    }
-
     func buildCollectionView() -> UICollectionView {
         let flowLayout = UICollectionViewFlowLayout()
         flowLayout.minimumInteritemSpacing = Constants.collectionViewPadding
@@ -380,26 +377,34 @@ fileprivate extension PaymentReviewViewController {
         let control = UIPageControl()
         control.pageIndicatorTintColor = model.configuration.pageIndicatorTintColor
         control.currentPageIndicatorTintColor = model.configuration.currentPageIndicatorTintColor
-        control.backgroundColor = model.configuration.backgroundColor
+        control.backgroundColor = .clear
         control.hidesForSinglePage = true
         control.numberOfPages = model.document?.pageCount ?? 0
         return control
     }
 
     func layoutContainerCollectionView() {
-        containerCollectionView.translatesAutoresizingMaskIntoConstraints = false
-        mainView.addSubview(containerCollectionView)
-        mainView.sendSubviewToBack(containerCollectionView)
+        collectionView.translatesAutoresizingMaskIntoConstraints = false
+        pageControl.translatesAutoresizingMaskIntoConstraints = false
+        mainView.addSubview(pageControl)
+        mainView.sendSubviewToBack(pageControl)
+
+        mainView.addSubview(collectionView)
+        mainView.sendSubviewToBack(collectionView)
+
+        let navigationBarHeight = self.navigationController?.navigationBar.frame.maxY ?? 0
 
         NSLayoutConstraint.activate([
-            containerCollectionView.leadingAnchor.constraint(equalTo: mainView.leadingAnchor),
-            containerCollectionView.trailingAnchor.constraint(equalTo: mainView.trailingAnchor),
-            containerCollectionView.topAnchor.constraint(equalTo: mainView.topAnchor),
-            containerCollectionView.bottomAnchor.constraint(equalTo: paymentInfoContainerView.topAnchor, constant: Constants.collectionViewBottomPadding),
+            collectionView.leadingAnchor.constraint(equalTo: mainView.leadingAnchor),
+            collectionView.trailingAnchor.constraint(equalTo: mainView.trailingAnchor),
+            collectionView.topAnchor.constraint(equalTo: mainView.topAnchor, constant: navigationBarHeight),
+            collectionView.bottomAnchor.constraint(equalTo: paymentInfoContainerView.topAnchor, constant: Constants.collectionViewBottomPadding),
 
             pageControl.heightAnchor.constraint(equalToConstant: Constants.pageControlHeight),
-            collectionView.widthAnchor.constraint(equalTo: containerCollectionView.widthAnchor),
-            collectionView.heightAnchor.constraint(equalTo: containerCollectionView.heightAnchor),
+            pageControl.bottomAnchor.constraint(equalTo: collectionView.bottomAnchor, constant: -Constants.collectionViewPadding),
+            pageControl.centerXAnchor.constraint(equalTo: mainView.centerXAnchor),
+            collectionView.widthAnchor.constraint(equalTo: collectionView.widthAnchor),
+            collectionView.heightAnchor.constraint(equalTo: collectionView.heightAnchor),
             paymentInfoContainerView.leadingAnchor.constraint(equalTo: view.leadingAnchor),
             paymentInfoContainerView.trailingAnchor.constraint(equalTo: view.trailingAnchor)
         ])

GiniComponents/GiniUtilites/Sources/GiniUtilites/Extensions/NSAttributedString.swift

@@ -8,8 +8,8 @@
 import UIKit
 
 extension NSMutableAttributedString {
-    public func addLinkToRange(link: String, range: NSRange, linkFont: UIFont, textToRemove: String?) {
-        var attributes: [NSAttributedString.Key: Any] = [.font: linkFont]
+    public func addLinkToRange(link: String, color: UIColor, range: NSRange, linkFont: UIFont, textToRemove: String?) {
+        var attributes: [NSAttributedString.Key: Any] = [.font: linkFont, .foregroundColor: color]
         if range.length > 0, let url = URL(string: link) {
             attributes[.link] = url
             self.addAttributes(attributes, range: range)

HealthSDK/GiniHealthSDK/Sources/GiniHealthSDK/Core/PaymentComponentsController+Helpers.swift

@@ -154,7 +154,6 @@ extension PaymentComponentsController {
         if previousPresentedViews.count > 0, previousPresentedViews.first != .paymentReview {
             previousPresentedViews.removeAll()
         }
-        previousPresentedViews.insert(.bankPicker)
         let paymentProvidersBottomViewModel = BanksBottomViewModel(paymentProviders: paymentProviders,
                                                                    selectedPaymentProvider: healthSelectedPaymentProvider,
                                                                    configuration: configurationProvider.bankSelectionConfiguration,
@@ -182,7 +181,8 @@ extension PaymentComponentsController {
      */
     func loadPaymentReviewScreenFor(trackingDelegate: GiniHealthTrackingDelegate?,
                                     completion: @escaping (UIViewController?, GiniHealthError?) -> Void) {
-        previousPresentedViews.insert(.paymentReview)
+        previousPresentedViews.append(.paymentReview)
+        let previousPaymentComponentScreenType: PaymentComponentScreenType? = previousPresentedViews.contains(.bankPicker) ? .bankPicker : nil
         if !GiniHealthConfiguration.shared.useInvoiceWithoutDocument {
             guard let documentId else {
                 completion(nil, nil)
@@ -197,26 +197,35 @@ extension PaymentComponentsController {
                             completion(nil, nil)
                             return
                         }
-                        self.preparePaymentReviewViewController(data: data, paymentInfo: nil, completion: completion)
+                        self.preparePaymentReviewViewController(data: data,
+                                                                paymentInfo: nil,
+                                                                previousPaymentComponentScreenType: previousPaymentComponentScreenType,
+                                                                completion: completion)
                     case .failure(let error):
                         completion(nil, error)
                 }
             }
         } else {
-            loadPaymentReviewScreenWithoutDocument(paymentInfo: paymentInfo, trackingDelegate: trackingDelegate, completion: completion)
+            loadPaymentReviewScreenWithoutDocument(paymentInfo: paymentInfo,
+                                                   trackingDelegate: trackingDelegate,
+                                                   previousPaymentComponentScreenType: previousPaymentComponentScreenType,
+                                                   completion: completion)
         }
     }
 
     private func loadPaymentReviewScreenWithoutDocument(paymentInfo: GiniInternalPaymentSDK.PaymentInfo?,
                                                         trackingDelegate: GiniHealthTrackingDelegate?,
+                                                        previousPaymentComponentScreenType: PaymentComponentScreenType? = nil,
                                                         completion: @escaping (UIViewController?, GiniHealthError?) -> Void) {
         preparePaymentReviewViewController(data: nil,
                                            paymentInfo: paymentInfo,
+                                           previousPaymentComponentScreenType: previousPaymentComponentScreenType,
                                            completion: completion)
     }
 
     private func preparePaymentReviewViewController(data: DataForReview?,
                                                     paymentInfo: GiniInternalPaymentSDK.PaymentInfo?,
+                                                    previousPaymentComponentScreenType: PaymentComponentScreenType? = nil,
                                                     completion: @escaping (UIViewController?, GiniHealthError?) -> Void) {
         guard let healthSelectedPaymentProvider else {
             completion(nil, nil)
@@ -240,7 +249,8 @@ extension PaymentComponentsController {
                                            poweredByGiniConfiguration: configurationProvider.poweredByGiniConfiguration,
                                            poweredByGiniStrings: stringsProvider.poweredByGiniStrings,
                                            bottomSheetConfiguration: configurationProvider.bottomSheetConfiguration,
-                                           showPaymentReviewCloseButton: configurationProvider.showPaymentReviewCloseButton)
+                                           showPaymentReviewCloseButton: configurationProvider.showPaymentReviewCloseButton,
+                                           previousPaymentComponentScreenType: previousPaymentComponentScreenType)
 
         let vc = PaymentReviewViewController.instantiate(viewModel: viewModel,
                                                          selectedPaymentProvider: healthSelectedPaymentProvider)
@@ -309,6 +319,7 @@ extension PaymentComponentsController {
         shareInvoiceBottomViewModel.viewDelegate = self
         shareInvoiceBottomViewModel.documentId = documentId
         let shareInvoiceBottomView = ShareInvoiceBottomView(viewModel: shareInvoiceBottomViewModel, bottomSheetConfiguration: configurationProvider.bottomSheetConfiguration)
+        shareInvoiceBottomSheet = shareInvoiceBottomView
         return shareInvoiceBottomView
     }
 
@@ -324,6 +335,18 @@ extension PaymentComponentsController {
             storeDefaultPaymentProvider(paymentProvider: provider)
         }
     }
+
+    /**
+     Notifies the controller that the Payment Review Screen has closed, enabling the cleanup of stored data used to manage various flows.
+     */
+    public func paymentReviewClosed(with previousPresentedView: PaymentComponentScreenType?) {
+        shareInvoiceBottomSheet = nil
+        if previousPresentedView == .bankPicker {
+            previousPresentedViews.append(.bankPicker)
+        } else {
+            previousPresentedViews.removeAll()
+        }
+    }
 
     /**
      Opens the more information view controller by notifying the view delegate. This method is used when opening the More Information screen inside the bank selection bottom sheet that's presented in the Payment Review Screen.
@@ -399,7 +422,12 @@ extension PaymentComponentsController {
                 return
             }
 
-            self?.sharePDF(pdfURL: pdfPath, paymentRequestId: paymentRequestId, viewController: viewController) { [weak self] (activity, _, _, _) in
+            self?.sharePDF(pdfURL: pdfPath, paymentRequestId: paymentRequestId, viewController: viewController) { [weak self] (activity, actionOnShareSheet, _, _) in
+                if !actionOnShareSheet {
+                    guard let shareInvoiceBottomSheet = self?.shareInvoiceBottomSheet else { return }
+                    self?.dismissAndPresent(viewController: shareInvoiceBottomSheet, animated: false)
+                }
+
                 guard activity != nil else {
                     return
                 }

HealthSDK/GiniHealthSDK/Sources/GiniHealthSDK/Core/PaymentComponentsController.swift

@@ -27,7 +27,9 @@ protocol PaymentComponentsProtocol {
     func checkIfDocumentIsPayable(docId: String, completion: @escaping (Result<Bool, GiniHealthError>) -> Void)
     func paymentView() -> UIView
     func bankSelectionBottomSheet() -> UIViewController
-    func loadPaymentReviewScreenFor(trackingDelegate: GiniHealthTrackingDelegate?, completion: @escaping (UIViewController?, GiniHealthError?) -> Void)
+    func loadPaymentReviewScreenFor(trackingDelegate: GiniHealthTrackingDelegate?,
+                                    previousPaymentComponentScreenType: PaymentComponentScreenType?,
+                                    completion: @escaping (UIViewController?, GiniHealthError?) -> Void)
     func paymentInfoViewController() -> UIViewController
     func paymentViewBottomSheet() -> UIViewController
 }
@@ -98,7 +100,7 @@ public final class PaymentComponentsController: BottomSheetsProviderProtocol, Gi
     }
 
     /// Previous presented view
-    var previousPresentedViews: Set<PaymentComponentScreenType> = []
+    var previousPresentedViews: [PaymentComponentScreenType] = []
     // Client's navigation controller provided in order to handle all HealthSDK flows
     weak var navigationControllerProvided: UINavigationController?
     // Payment Information from the invoice that contains a document or not
@@ -107,7 +109,9 @@ public final class PaymentComponentsController: BottomSheetsProviderProtocol, Gi
     var documentId: String?
     // Errors stack received from API. We will show them for the clients
     var errors: [String] = []
-
+
+    // Store Share Bottom Sheet for dismissed native share modal
+    var shareInvoiceBottomSheet: ShareInvoiceBottomView?
     /**
      Initializer of the Payment Component Controller class.
 

HealthSDK/GiniHealthSDK/Sources/GiniHealthSDK/Resources/en.lproj/Localizable.strings

@@ -36,7 +36,7 @@
 "gini.health.paymentcomponent.payment.info.pay.bills.title.label" = "Pay bills easily with the banking app.";
 "gini.health.paymentcomponent.payment.info.pay.bills.description.label" = "Medical bills and other submitted receipts can now be paid very easily.\nThe payment data such as IBAN, amount, recipient and purpose are seamlessly transferred to the banking app, and the payment only needs to be confirmed there.\nYou can also park the invoice and pay it within 3 months after uploading.\nThe data is encrypted and transferred securely to your banking app. The data protection regulations of your bank apply.\nSupported by the largest banks. Integration by Gini[LINK].";
 "gini.health.paymentcomponent.payment.info.pay.bills.description.clickable.text" = "Gini[LINK]";
-"gini.health.paymentcomponent.paymentinfo.questions.title.label" = "Frequently asked questions";
+"gini.health.paymentcomponent.payment.info.questions.title.label" = "Frequently Asked Questions";
 "gini.health.paymentcomponent.payment.info.questions.question.1" = "Can I submit invoices and pay them later?";
 "gini.health.paymentcomponent.payment.info.questions.question.2" = "Is the service free of charge?";
 "gini.health.paymentcomponent.payment.info.questions.question.3" = "Is my data secure?";