Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-11-02 20:16:04 +01:00
Code Issues Packages Projects Releases Wiki Activity

feat: change chat notification delivery

Browse Source 
instead of checking if user is online in the past x minutes, check if the user has read the room, if they have already marked the chat as read don't send notification
This commit is contained in:
Barış Soner Uşaklı
2023-03-26 21:13:07 -04:00
parent 2a9b3ee464
commit 2c5a5bf366
2 changed files with 12 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/messaging/notifications.js
View File

@@ -57,8 +57,9 @@ module.exports = function (Messaging) {
};
async function sendNotifications(fromuid, uids, roomId, messageObj) {
const isOnline = await user.isOnline(uids);
uids = uids.filter((uid, index) => !isOnline[index] && parseInt(fromuid, 10) !== parseInt(uid, 10));
const hasRead = await Messaging.hasRead(uids, roomId);
uids = uids.filter((uid, index) => !hasRead[index] && parseInt(fromuid, 10) !== parseInt(uid, 10));
console.log('checking notif', uids, hasRead);
if (!uids.length) {
delete Messaging.notifyQueue[`${fromuid}:${roomId}`];
return;