Merge pull request #40623 from bernhardoj/fix/39677-missing-translati…

…on-notification-preference Fix missing translation shown in notification preference
Expensify · Apr 23, 2024 · c90b46e · c90b46e
2 parents ffd468b + 1502ffc
commit c90b46e
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/pages/ProfilePage.tsx b/src/pages/ProfilePage.tsx
@@ -130,7 +130,8 @@ function ProfilePage({route}: ProfilePageProps) {
 
     const navigateBackTo = route?.params?.backTo;
 
-    const shouldShowNotificationPreference = !isEmptyObject(report) && !isCurrentUser && report.notificationPreference !== CONST.REPORT.NOTIFICATION_PREFERENCE.HIDDEN;
+    const shouldShowNotificationPreference =
+        !isEmptyObject(report) && !isCurrentUser && !!report.notificationPreference && report.notificationPreference !== CONST.REPORT.NOTIFICATION_PREFERENCE.HIDDEN;
     const notificationPreference = shouldShowNotificationPreference
         ? translate(`notificationPreferencesPage.notificationPreferences.${report.notificationPreference}` as TranslationPaths)
         : '';
@@ -236,7 +237,7 @@ function ProfilePage({route}: ProfilePageProps) {
                                     shouldShowRightIcon
                                 />
                             )}
-                            {!isEmptyObject(report) && !isCurrentUser && (
+                            {!isEmptyObject(report) && report.reportID && !isCurrentUser && (
                                 <MenuItem
                                     title={`${translate('privateNotes.title')}`}
                                     titleStyle={styles.flex1}