Merge pull request #4153 from Northeastern-Electric-Racing/4090-maintenance---schema-changes-move-notifications-to-event-level

4090 maintenance   schema changes move notifications to event level

Author: wavehassman

src/backend/src/controllers/calendar.controllers.ts

@@ -21,8 +21,7 @@ export default class CalendarController {
         documents,
         description,
         onlyHeadsOrAbove,
-        requiresConfirmation,
-        sendSlackNotifications
+        requiresConfirmation
       } = req.body;
 
       const eventType = await CalendarService.createEventType(
@@ -43,8 +42,7 @@ export default class CalendarController {
         documents,
         description,
         onlyHeadsOrAbove,
-        requiresConfirmation,
-        sendSlackNotifications
+        requiresConfirmation
       );
       res.status(200).json(eventType);
     } catch (error: unknown) {
@@ -205,8 +203,7 @@ export default class CalendarController {
         documents,
         description,
         onlyHeadsOrAbove,
-        requiresConfirmation,
-        sendSlackNotifications
+        requiresConfirmation
       } = req.body;
 
       const eventType = await CalendarService.editEventType(
@@ -228,8 +225,7 @@ export default class CalendarController {
         documents,
         description,
         onlyHeadsOrAbove,
-        requiresConfirmation,
-        sendSlackNotifications
+        requiresConfirmation
       );
       res.status(200).json(eventType);
     } catch (error: unknown) {
@@ -278,7 +274,8 @@ export default class CalendarController {
         questionDocumentLink,
         location,
         zoomLink,
-        description
+        description,
+        sendSlackNotifications
       } = req.body;
 
       const parsedScheduleSlots = scheduleSlots.map((slot: any) => ({
@@ -302,6 +299,7 @@ export default class CalendarController {
         workPackageIds,
         parsedScheduleSlots,
         parsedInitialDateScheduled,
+        sendSlackNotifications,
         teamTypeId,
         questionDocumentLink,
         location,
@@ -332,7 +330,8 @@ export default class CalendarController {
         questionDocumentLink,
         location,
         zoomLink,
-        description
+        description,
+        sendSlackNotifications
       } = req.body;
 
       const event = await CalendarService.editEvent(
@@ -348,6 +347,7 @@ export default class CalendarController {
         machineryIds,
         workPackageIds,
         documents,
+        sendSlackNotifications,
         teamTypeId,
         questionDocumentLink,
         location,

src/backend/src/prisma/migrations/20260416232819_move_send_slack_notifs_to_event/migration.sql

@@ -0,0 +1,12 @@
+-- AlterTable
+ALTER TABLE "Event" ADD COLUMN     "sendSlackNotifications" BOOLEAN NOT NULL DEFAULT false;
+
+-- Backfill Event.sendSlackNotifications from Event_Type before dropping the old column
+UPDATE "Event" e
+SET "sendSlackNotifications" = true
+FROM "Event_Type" et
+WHERE e."eventTypeId" = et."eventTypeId"
+AND et."sendSlackNotifications" = true;
+
+-- AlterTable
+ALTER TABLE "Event_Type" DROP COLUMN "sendSlackNotifications";

src/backend/src/prisma/schema.prisma

@@ -1180,6 +1180,7 @@ model Event {
   description                String?
   notificationSlackThreads   Message_Info[]
   calendarEventIds           String[]
+  sendSlackNotifications     Boolean         @default(false)
 }
 
 model Calendar {
@@ -1222,7 +1223,6 @@ model Event_Type {
   description                      Boolean
   onlyHeadsOrAboveForEventCreation Boolean
   requiresConfirmation             Boolean
-  sendSlackNotifications           Boolean
   events                           Event[]
   organizationId                   String
   organization                     Organization @relation(fields: [organizationId], references: [organizationId])

src/backend/src/prisma/seed.ts

@@ -4806,7 +4806,6 @@ const performSeed: () => Promise<void> = async () => {
     false,
     true,
     false,
-    false,
     false
   );
 
@@ -4829,7 +4828,6 @@ const performSeed: () => Promise<void> = async () => {
     true,
     true,
     false,
-    true,
     true
   );
 
@@ -4852,7 +4850,6 @@ const performSeed: () => Promise<void> = async () => {
     true,
     false,
     false,
-    false,
     false
   );
 
@@ -4875,7 +4872,6 @@ const performSeed: () => Promise<void> = async () => {
     false,
     false,
     false,
-    false,
     false
   );
 
@@ -4898,6 +4894,7 @@ const performSeed: () => Promise<void> = async () => {
       }
     ],
     undefined,
+    true,
     mechanical.teamTypeId,
     undefined,
     'Conference Room A',
@@ -4939,6 +4936,7 @@ const performSeed: () => Promise<void> = async () => {
       }
     ],
     undefined,
+    false,
     mechanical.teamTypeId,
     undefined,
     'Conference Room A',
@@ -4965,6 +4963,7 @@ const performSeed: () => Promise<void> = async () => {
       }
     ],
     undefined,
+    false,
     mechanical.teamTypeId,
     undefined,
     'Conference Room A',
@@ -4991,6 +4990,7 @@ const performSeed: () => Promise<void> = async () => {
       }
     ],
     undefined,
+    true,
     mechanical.teamTypeId,
     undefined,
     'Conference Room A',
@@ -5017,6 +5017,7 @@ const performSeed: () => Promise<void> = async () => {
       }
     ],
     undefined,
+    true,
     mechanical.teamTypeId,
     undefined,
     'Conference Room A',
@@ -5043,6 +5044,7 @@ const performSeed: () => Promise<void> = async () => {
       }
     ],
     undefined,
+    false,
     mechanical.teamTypeId,
     undefined,
     'Conference Room A',
@@ -5063,6 +5065,7 @@ const performSeed: () => Promise<void> = async () => {
     [workPackage3.id],
     [],
     weeksFromNow(1),
+    true,
     software.teamTypeId,
     'https://docs.google.com/document/d/2_example',
     'Conference Room B',
@@ -5089,6 +5092,7 @@ const performSeed: () => Promise<void> = async () => {
       }
     ],
     undefined,
+    true,
     electrical.teamTypeId,
     'https://docs.google.com/document/d/3_example',
     undefined,
@@ -5115,6 +5119,7 @@ const performSeed: () => Promise<void> = async () => {
       }
     ],
     undefined,
+    false,
     mechanical.teamTypeId,
     undefined,
     undefined,

src/backend/src/routes/calendar.routes.ts

@@ -64,7 +64,6 @@ calendarRouter.post(
   body('description').isBoolean(),
   body('onlyHeadsOrAbove').isBoolean(),
   body('requiresConfirmation').isBoolean(),
-  body('sendSlackNotifications').isBoolean(),
   validateInputs,
   CalendarController.createEventType
 );
@@ -88,7 +87,6 @@ calendarRouter.post(
   body('description').isBoolean(),
   body('onlyHeadsOrAbove').isBoolean(),
   body('requiresConfirmation').isBoolean(),
-  body('sendSlackNotifications').isBoolean(),
   validateInputs,
   CalendarController.editEventType
 );
@@ -119,6 +117,7 @@ calendarRouter.post(
   isDate(body('scheduleSlots.*.startTime')),
   isDate(body('scheduleSlots.*.endTime')),
   body('scheduleSlots.*.allDay').isBoolean(),
+  body('sendSlackNotifications').isBoolean(),
   validateInputs,
   CalendarController.createEvent
 );
@@ -147,6 +146,7 @@ calendarRouter.post(
   nonEmptyString(body('documents.*.googleFileId')),
   body('questionDocumentLink').optional().isString(),
   body('description').optional().isString(),
+  body('sendSlackNotifications').isBoolean(),
   validateInputs,
   CalendarController.editEvent
 );