Merged PR 33332: Remove alert when succesfully logged in in city-pass…

… module Remove alert when succesfully logged in in city-pass module Related work items: #126268
Amsterdam · Sep 19, 2024 · 351ffe7 · 351ffe7
2 parents 560c7fc + 7a64817
commit 351ffe7
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 19 deletions.
diff --git a/src/modules/city-pass/alerts.ts b/src/modules/city-pass/alerts.ts
@@ -3,22 +3,22 @@ import {
   AlertVariant,
 } from '@/components/ui/feedback/alert/Alert.types'
 
+const LOGOUT_ALERT = {
+  text: 'Je Stadspas gegevens zijn niet meer zichtbaar in de app. Je kunt je Stadspas gegevens altijd weer zien door in te loggen.',
+  title: 'Uitgelogd',
+  hasIcon: true,
+  hasCloseIcon: true,
+  testID: 'CityPassLoggedOutAlert',
+}
+
 export const alerts = {
   logoutWarning: {
     variant: AlertVariant.warning,
-    text: 'Je Stadspas gegevens zijn niet meer zichtbaar in de app. Je kunt je Stadspas gegevens altijd weer zien door in te loggen.',
-    title: 'Uitgelogd',
-    hasIcon: true,
-    hasCloseIcon: true,
-    testID: 'CityPassLoggedOutAlert',
+    ...LOGOUT_ALERT,
   },
   logoutSuccess: {
     variant: AlertVariant.positive,
-    text: 'Je Stadspas gegevens zijn niet meer zichtbaar in de app. Je kunt je Stadspas gegevens altijd weer zien door in te loggen.',
-    title: 'Uitgelogd',
-    hasIcon: true,
-    hasCloseIcon: true,
-    testID: 'CityPassLoggedOutAlert',
+    ...LOGOUT_ALERT,
   },
   logoutFailed: {
     variant: AlertVariant.negative,
@@ -36,14 +36,6 @@ export const alerts = {
     hasCloseIcon: true,
     testID: 'CityPassLoggedInAlertNegative',
   },
-  loginSuccess: {
-    variant: AlertVariant.positive,
-    text: 'Je Stadspas staat nu ook in de app.',
-    title: 'Gelukt!',
-    hasIcon: true,
-    hasCloseIcon: true,
-    testID: 'CityPassLoggedInAlertPositive',
-  },
 } as const satisfies AlertsRecord
 
 export type Alerts = keyof typeof alerts
diff --git a/src/modules/city-pass/screens/Dashboard.screen.tsx b/src/modules/city-pass/screens/Dashboard.screen.tsx
@@ -44,7 +44,6 @@ export const DashboardScreen = ({navigation, route}: Props) => {
   useEffect(() => {
     if (loginResult === LoginResult.success) {
       dispatch(setIsCityPassOwnerRegistered(true))
-      setAlert(alerts.loginSuccess)
 
       if (deeplinkAccessToken && deeplinkRefreshToken) {
         void setSecureItem(