From 5aabc8a3c384f208ed156345f3ba91e9e536609f Mon Sep 17 00:00:00 2001
From: Chris Kosuke Tseng <ctseng@expensify.com>
Date: Fri, 3 Jun 2022 10:09:52 -0700
Subject: [PATCH] Merge pull request #9291 from
 Expensify/amal-decouple-betas-default-policy

Decouple policyRoom and defaultRoom betas

(cherry picked from commit 994841f472545dedace6ae9c36dd573fec63b2d5)
---
 src/libs/OptionsListUtils.js                | 12 ++++++++----
 src/pages/home/ReportScreen.js              |  6 +++++-
 src/pages/workspace/WorkspaceNewRoomPage.js |  4 ++--
 3 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/src/libs/OptionsListUtils.js b/src/libs/OptionsListUtils.js
index 339dd2c6e25..5c62eb336ed 100644
--- a/src/libs/OptionsListUtils.js
+++ b/src/libs/OptionsListUtils.js
@@ -365,7 +365,7 @@ function getOptions(reports, personalDetails, activeReportID, {
     selectedOptions = [],
     maxRecentReportsToShow = 0,
     excludeLogins = [],
-    excludeDefaultRooms = false,
+    excludeChatRooms = false,
     includeMultipleParticipantReports = false,
     includePersonalDetails = false,
     includeRecentReports = false,
@@ -439,11 +439,11 @@ function getOptions(reports, personalDetails, activeReportID, {
             return;
         }
 
-        if (isChatRoom && (!Permissions.canUseDefaultRooms(betas) || excludeDefaultRooms)) {
+        if (isChatRoom && excludeChatRooms) {
             return;
         }
 
-        if (isPolicyExpenseChat && !Permissions.canUsePolicyExpenseChat(betas)) {
+        if (ReportUtils.isDefaultRoom(report) && !Permissions.canUseDefaultRooms(betas)) {
             return;
         }
 
@@ -451,6 +451,10 @@ function getOptions(reports, personalDetails, activeReportID, {
             return;
         }
 
+        if (isPolicyExpenseChat && !Permissions.canUsePolicyExpenseChat(betas)) {
+            return;
+        }
+
         // Save the report in the map if this is a single participant so we can associate the reportID with the
         // personal detail option later. Individuals should not be associated with single participant
         // policyExpenseChats or chatRooms since those are not people.
@@ -703,7 +707,7 @@ function getNewChatOptions(
         betas,
         searchValue: searchValue.trim(),
         selectedOptions,
-        excludeDefaultRooms: true,
+        excludeChatRooms: true,
         includeRecentReports: true,
         includePersonalDetails: true,
         maxRecentReportsToShow: 5,
diff --git a/src/pages/home/ReportScreen.js b/src/pages/home/ReportScreen.js
index e8ebd17d411..ced4faa2bca 100644
--- a/src/pages/home/ReportScreen.js
+++ b/src/pages/home/ReportScreen.js
@@ -159,7 +159,11 @@ class ReportScreen extends React.Component {
             return null;
         }
 
-        if (!Permissions.canUseDefaultRooms(this.props.betas) && ReportUtils.isChatRoom(this.props.report)) {
+        if (!Permissions.canUseDefaultRooms(this.props.betas) && ReportUtils.isDefaultRoom(this.props.report)) {
+            return null;
+        }
+
+        if (!Permissions.canUsePolicyRooms(this.props.betas) && ReportUtils.isUserCreatedPolicyRoom(this.props.report)) {
             return null;
         }
 
diff --git a/src/pages/workspace/WorkspaceNewRoomPage.js b/src/pages/workspace/WorkspaceNewRoomPage.js
index 6aec1b883eb..5740c74ae2e 100644
--- a/src/pages/workspace/WorkspaceNewRoomPage.js
+++ b/src/pages/workspace/WorkspaceNewRoomPage.js
@@ -137,8 +137,8 @@ class WorkspaceNewRoomPage extends React.Component {
     }
 
     render() {
-        if (!Permissions.canUseDefaultRooms(this.props.betas)) {
-            Log.info('Not showing create Policy Room page since user is not on default rooms beta');
+        if (!Permissions.canUsePolicyRooms(this.props.betas)) {
+            Log.info('Not showing create Policy Room page since user is not on policy rooms beta');
             Navigation.dismissModal();
             return null;
         }