Remove notification batching & add specification

Instead of /notification/batch -> /notification/send just do /notification/send since we are moving from serverless Also changes `GET /notifications/`` response type from `[{..notif1}]` to `{ notifications: [{..notif1}] }` to be consistent
HackIllinois · Oct 16, 2024 · 7671e9e · 7671e9e
1 parent 27579bd
commit 7671e9e
Show file tree

Hide file tree

Showing 6 changed files with 223 additions and 231 deletions.
diff --git a/src/common/config.ts b/src/common/config.ts
@@ -138,7 +138,6 @@ const Config = {
     /* Limits */
     LEADERBOARD_QUERY_LIMIT: 25,
     MAX_RESUME_SIZE_BYTES: 2 * 1024 * 1024,
-    NOTIFICATION_BATCH_SIZE: 50,
 
     /* Misc */
     SHOP_BYTES_GEN: 2,

diff --git a/src/database/models.ts b/src/database/models.ts
@@ -12,7 +12,7 @@ import { ShopItem } from "./shop-db";
 import { UserAttendance, UserInfo } from "../services/user/user-schemas";
 import { AnyParamConstructor, IModelOptions } from "@typegoose/typegoose/lib/types";
 import { StaffShift } from "../services/staff/staff-schemas";
-import { NotificationMappings, NotificationMessages } from "./notification-db";
+import { NotificationMappings, NotificationMessages } from "../services/notification/notification-schemas";
 import { PuzzleItem } from "./puzzle-db";
 
 // Groups for collections

diff --git a/src/database/notification-db.ts b/src/database/notification-db.ts
diff --git a/src/services/notification/notification-formats.ts b/src/services/notification/notification-formats.ts