Merge pull request #1200 from HedvigInsurance/fix/edit-coinsured/dism…

…issing-view-on-canceleing-edit-single-co-insured Edit co insured journey
HedvigInsurance · Jan 24, 2024 · ef61f07 · ef61f07
2 parents 4a5e4af + 66784f5
commit ef61f07
Showing 1 changed file with 14 additions and 7 deletions.
diff --git a/Projects/EditCoInsured/Sources/Journeys/EditCoInsuredJourney.swift b/Projects/EditCoInsured/Sources/Journeys/EditCoInsuredJourney.swift
@@ -48,7 +48,7 @@ public class EditCoInsuredJourney {
             getScreen(for: action)
         }
         .configureTitle(L10n.coinsuredEditTitle)
-        .withJourneyDismissButton
+        .addDismissEditCoInsuredFlow()
     }
 
     static func openNewInsuredPeopleScreen(config: InsuredPeopleConfig) -> some JourneyPresentation {
@@ -63,7 +63,7 @@ public class EditCoInsuredJourney {
             getScreen(for: action)
         }
         .configureTitle(L10n.coinsuredEditTitle)
-        .withJourneyDismissButton
+        .addDismissEditCoInsuredFlow()
     }
 
     @JourneyBuilder
@@ -263,11 +263,7 @@ public class EditCoInsuredJourney {
             style: .detented(.scrollViewContentSize),
             options: [.largeNavigationBar, .blurredBackground]
         ) { action in
-            if case .coInsuredNavigationAction(action: .dismissEdit) = action {
-                PopJourney()
-            } else {
-                getScreen(for: action)
-            }
+            getScreen(for: action)
         }
         .configureTitle(L10n.SelectInsurance.NavigationBar.CenterElement.title)
     }
@@ -335,3 +331,14 @@ public class EditCoInsuredJourney {
         }
     }
 }
+
+extension JourneyPresentation {
+    func addDismissEditCoInsuredFlow() -> some JourneyPresentation {
+        self.withJourneyDismissButtonWithConfirmation(
+            withTitle: L10n.General.areYouSure,
+            andBody: L10n.Claims.Alert.body,
+            andCancelText: L10n.General.no,
+            andConfirmText: L10n.General.yes
+        )
+    }
+}