ekzyis/stacker.news
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Include name in mention push notification (#2045)

Browse Source
This commit is contained in:
ekzyis 2025-03-31 11:59:57 -05:00 committed by GitHub
parent 3568fc1c62
commit fca4d1ff92
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 10 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
api/paidAction/itemUpdate.js
View File

@ -163,7 +163,8 @@ export async function nonCriticalSideEffects ({ invoice, id }, { models }) {
where: invoice ? { invoiceId: invoice.id } : { id: parseInt(id) },
include: {
mentions: true,
itemReferrers: { include: { refereeItem: true } }
itemReferrers: { include: { refereeItem: true } },
user: true
}
})
// compare timestamps to only notify if mention or item referral was just created to avoid duplicates on edits

16
lib/webPush.js
View File

@ -319,14 +319,14 @@ export const notifyZapped = async ({ models, item }) => {
export const notifyMention = async ({ models, userId, item }) => {
try {
const muted = await isMuted({ models, muterId: userId, mutedId: item.userId })
if (!muted) {
await sendUserNotification(userId, {
title: 'you were mentioned',
body: item.text,
item,
tag: 'MENTION'
})
}
if (muted) return
await sendUserNotification(userId, {
title: `@${item.user.name} mentioned you`,
body: item.text,
item,
tag: 'MENTION'
})
} catch (err) {
console.error(err)
}