Skip to content

Commit 6b0b4f9

Browse files
authored
Merge pull request Expensify#68035 from Eskalifer1/fix/67694
fix:67694:use same notification format for group chats as for other chat rooms
2 parents b6ea9a1 + 467cc6e commit 6b0b4f9

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/libs/Notification/LocalNotification/BrowserNotifications.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ export default {
100100
let body;
101101
const icon = usesIcon ? EXPENSIFY_ICON_URL : '';
102102

103-
const isChatRoom = ReportUtils.isChatRoom(report) || ReportUtils.isPolicyExpenseChat(report);
103+
const isRoomOrGroupChat = ReportUtils.isChatRoom(report) || ReportUtils.isPolicyExpenseChat(report) || ReportUtils.isGroupChat(report);
104104

105105
const {person, message} = reportAction;
106106
const plainTextPerson = person?.map((f) => Str.removeSMSDomain(f.text ?? '')).join() ?? '';
@@ -113,7 +113,7 @@ export default {
113113
plainTextMessage = message?.type === 'COMMENT' ? getTextFromHtml(message?.html) : '';
114114
}
115115

116-
if (isChatRoom) {
116+
if (isRoomOrGroupChat) {
117117
const roomName = ReportUtils.getReportName(report);
118118
title = roomName;
119119
body = `${plainTextPerson}: ${plainTextMessage}`;

0 commit comments

Comments
 (0)