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

feat: keep notifs for one month, load 50 notifications instead of 30

Browse Source
This commit is contained in:
Barış Soner Uşaklı
2021-02-26 22:22:12 -05:00
parent 2674de01dd
commit 02f08111cf
2 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/notifications.js
View File

@@ -318,8 +318,8 @@ Notifications.markAllRead = async function (uid) {
};
Notifications.prune = async function () {
const week = 604800000;
const cutoffTime = Date.now() - week;
const month = 2592000000;
const cutoffTime = Date.now() - month;
const nids = await db.getSortedSetRangeByScore('notifications', 0, 500, '-inf', cutoffTime);
if (!nids.length) {
return;

6
src/user/notifications.js
View File

@@ -18,11 +18,11 @@ UserNotifications.get = async function (uid) {
return { read: [], unread: [] };
}
let unread = await getNotificationsFromSet(`uid:${uid}:notifications:unread`, uid, 0, 29);
let unread = await getNotificationsFromSet(`uid:${uid}:notifications:unread`, uid, 0, 49);
unread = unread.filter(Boolean);
let read = [];
if (unread.length < 30) {
read = await getNotificationsFromSet(`uid:${uid}:notifications:read`, uid, 0, 29 - unread.length);
if (unread.length < 50) {
read = await getNotificationsFromSet(`uid:${uid}:notifications:read`, uid, 0, 49 - unread.length);
}
return await plugins.hooks.fire('filter:user.notifications.get', {