Merge pull request #41380 from GandalfGwaihir/issue41280

[Access control]: Hide options `visibility` and `room name` for invoices
Expensify · May 1, 2024 · 573d989 · 573d989
2 parents 5f50430 + 8b2815c
commit 573d989
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/pages/settings/Report/ReportSettingsPage.tsx b/src/pages/settings/Report/ReportSettingsPage.tsx
@@ -35,7 +35,7 @@ function ReportSettingsPage({report, policies}: ReportSettingsPageProps) {
     const isMoneyRequestReport = ReportUtils.isMoneyRequestReport(report);
 
     const shouldDisableSettings = isEmptyObject(report) || ReportUtils.isArchivedRoom(report) || ReportUtils.isSelfDM(report);
-    const shouldShowRoomName = !ReportUtils.isPolicyExpenseChat(report) && !ReportUtils.isChatThread(report);
+    const shouldShowRoomName = !ReportUtils.isPolicyExpenseChat(report) && !ReportUtils.isChatThread(report) && !ReportUtils.isInvoiceRoom(report);
     const notificationPreference =
         report?.notificationPreference && report.notificationPreference !== CONST.REPORT.NOTIFICATION_PREFERENCE.HIDDEN
             ? translate(`notificationPreferencesPage.notificationPreferences.${report.notificationPreference}`)
@@ -149,6 +149,7 @@ function ReportSettingsPage({report, policies}: ReportSettingsPageProps) {
                         )}
                     </View>
                     {!!report?.visibility &&
+                        report.chatType !== CONST.REPORT.CHAT_TYPE.INVOICE &&
                         (shouldAllowChangeVisibility ? (
                             <MenuItemWithTopDescription
                                 shouldShowRightIcon