diff options
| author | tamaina <tamaina@hotmail.co.jp> | 2022-06-12 19:28:13 +0900 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-06-12 19:28:13 +0900 |
| commit | 11a6bd890cc752ee7abd2890773054743433eae1 (patch) | |
| tree | 06caab47ade7484de8fd933f61aa316946e6252a /packages/backend/src/server/api | |
| parent | 12.111.0 (diff) | |
| download | sharkey-11a6bd890cc752ee7abd2890773054743433eae1.tar.gz sharkey-11a6bd890cc752ee7abd2890773054743433eae1.tar.bz2 sharkey-11a6bd890cc752ee7abd2890773054743433eae1.zip | |
fix: some fixes of multiple notification read (#8819)
* fix: limit multiple notification read
* fix
* fix
Diffstat (limited to 'packages/backend/src/server/api')
| -rw-r--r-- | packages/backend/src/server/api/common/read-notification.ts | 2 | ||||
| -rw-r--r-- | packages/backend/src/server/api/endpoints/notifications/read.ts | 6 |
2 files changed, 7 insertions, 1 deletions
diff --git a/packages/backend/src/server/api/common/read-notification.ts b/packages/backend/src/server/api/common/read-notification.ts index 0dad35bcc2..8c4ba41a36 100644 --- a/packages/backend/src/server/api/common/read-notification.ts +++ b/packages/backend/src/server/api/common/read-notification.ts @@ -9,6 +9,8 @@ export async function readNotification( userId: User['id'], notificationIds: Notification['id'][] ) { + if (notificationIds.length === 0) return; + // Update documents await Notifications.update({ id: In(notificationIds), diff --git a/packages/backend/src/server/api/endpoints/notifications/read.ts b/packages/backend/src/server/api/endpoints/notifications/read.ts index 65e96d4862..e7839b2460 100644 --- a/packages/backend/src/server/api/endpoints/notifications/read.ts +++ b/packages/backend/src/server/api/endpoints/notifications/read.ts @@ -34,7 +34,11 @@ export const paramDef = { { type: 'object', properties: { - notificationIds: { type: 'array', items: { type: 'string', format: 'misskey:id' } }, + notificationIds: { + type: 'array', + items: { type: 'string', format: 'misskey:id' }, + maxItems: 100, + }, }, required: ['notificationIds'], }, |