Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-26 08:36:12 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: handle public chat rooms too

Browse Source
This commit is contained in:
Julian Lam
2023-10-25 12:11:37 -04:00
parent 18c27d1ad3
commit 6a696c43a3
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/messaging/notifications.js
View File

@@ -70,13 +70,13 @@ module.exports = function (Messaging) {
}
try {
await sendNotification(fromUid, roomId, messageObj);
await sendNotification(fromUid, roomId, isPublic, messageObj);
} catch (err) {
winston.error(`[messaging/notifications] Unabled to send notification\n${err.stack}`);
}
};
async function sendNotification(fromUid, roomId, messageObj) {
async function sendNotification(fromUid, roomId, isPublic, messageObj) {
fromUid = parseInt(fromUid, 10);
const [settings, roomData] = await Promise.all([
@@ -100,7 +100,7 @@ module.exports = function (Messaging) {
});
// Suppress notifications for users who are literally present in the room
const realtimeUids = await io.getUidsInRoom(`chat_room_${roomId}`);
const realtimeUids = await io.getUidsInRoom(`chat_room_${isPublic ? 'public_' : ''}${roomId}`);
uidsToNotify = uidsToNotify.filter(uid => !realtimeUids.includes(parseInt(uid, 10)));
if (uidsToNotify.length) {