Merge pull request #9758 from Expensify/Rory-StandardizeChatRoomName

Standardize on auth providing # prefix in chat rooms
Expensify · Jul 18, 2022 · 13d2766 · 13d2766
2 parents 93d7768 + 035ecc7
commit 13d2766
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 8 deletions.
diff --git a/src/libs/ReportUtils.js b/src/libs/ReportUtils.js
@@ -464,11 +464,7 @@ function getDisplayNamesWithTooltips(participants, isMultipleParticipantReport)
  */
 function getReportName(report, personalDetailsForParticipants = {}, policies = {}) {
     let formattedName;
-    if (isDefaultRoom(report)) {
-        formattedName = `#${report.reportName}`;
-    }
-
-    if (isUserCreatedPolicyRoom(report)) {
+    if (isChatRoom(report)) {
         formattedName = report.reportName;
     }
 

diff --git a/tests/unit/OptionsListUtilsTest.js b/tests/unit/OptionsListUtilsTest.js
@@ -807,7 +807,7 @@ describe('OptionsListUtils', () => {
                 participants: ['test3@instantworkspace.com'],
                 policyID: 'ABC123',
                 reportID: 12,
-                reportName: 'admins',
+                reportName: '#admins',
                 unreadActionCount: 0,
                 visibility: undefined,
             },
@@ -822,7 +822,7 @@ describe('OptionsListUtils', () => {
                 participants: ['test3@instantworkspace.com'],
                 policyID: 'ABC123',
                 reportID: 13,
-                reportName: 'admins',
+                reportName: '#admins',
                 unreadActionCount: 0,
                 visibility: undefined,
                 stateNum: CONST.REPORT.STATE_NUM.SUBMITTED,

diff --git a/tests/unit/ReportUtilsTest.js b/tests/unit/ReportUtilsTest.js
@@ -121,7 +121,7 @@ describe('ReportUtils', () => {
         describe('Default Policy Room', () => {
             const baseAdminsRoom = {
                 chatType: CONST.REPORT.CHAT_TYPE.POLICY_ADMINS,
-                reportName: 'admins',
+                reportName: '#admins',
             };
 
             test('Active', () => {