zio/bsky-app
2
0
Fork 0
Code Pull requests Releases Packages Activity

Group together notifications more aggressively

Browse source
This commit is contained in:
Paul Frazee 2023-01-17 16:25:10 -06:00
parent 5a5835f561
commit 90ac2fe1fc
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
src/state/models/notifications-view.ts
View file

@ -14,7 +14,8 @@ import {cleanError} from '../../lib/strings'
const UNGROUPABLE_REASONS = ['assertion'] const UNGROUPABLE_REASONS = ['assertion']
const PAGE_SIZE = 30 const PAGE_SIZE = 30
const MS_60MIN = 1e3 * 60 * 60 const MS_1HR = 1e3 * 60 * 60
const MS_2DAY = MS_1HR * 48
let _idCounter = 0 let _idCounter = 0
@ -447,7 +448,7 @@ function groupNotifications(
for (const item2 of items2) { for (const item2 of items2) {
const ts2 = +new Date(item2.indexedAt) const ts2 = +new Date(item2.indexedAt)
if ( if (
Math.abs(ts2 - ts) < MS_60MIN && Math.abs(ts2 - ts) < MS_2DAY &&
item.reason === item2.reason && item.reason === item2.reason &&
item.reasonSubject === item2.reasonSubject && item.reasonSubject === item2.reasonSubject &&
item.author.did !== item2.author.did item.author.did !== item2.author.did