natty
/
calckey
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

fix: Detach push notifications.. 

..from "isRead" check. Apps will handle that theself.
This commit is contained in:
April John 2023-05-08 11:36:32 +02:00
parent 8c6475961e
commit 0b59937e9e
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
packages/backend/src/services/create-notification.ts
View File

@ -80,6 +80,9 @@ export async function createNotification(
setTimeout(async () => { setTimeout(async () => {
const fresh = await Notifications.findOneBy({ id: notification.id }); const fresh = await Notifications.findOneBy({ id: notification.id });
if (fresh == null) return; // 既に削除されているかもしれない if (fresh == null) return; // 既に削除されているかもしれない
// We execute this before, because the server side "read" check doesnt work well with push notifications, the app and service worker will decide themself
// when it is best to show push notifications
pushNotification(notifieeId, "notification", packed);
if (fresh.isRead) return; if (fresh.isRead) return;
//#region ただしミュートしているユーザーからの通知なら無視 //#region ただしミュートしているユーザーからの通知なら無視
@ -95,7 +98,6 @@ export async function createNotification(
//#endregion //#endregion
publishMainStream(notifieeId, "unreadNotification", packed); publishMainStream(notifieeId, "unreadNotification", packed);
pushNotification(notifieeId, "notification", packed);
if (type === "follow") if (type === "follow")
sendEmailNotification.follow( sendEmailNotification.follow(