#4032 slack event msgs + admin tools disable w/ tooltip

Author: cielbellerose

src/backend/src/services/notifications.services.ts

@@ -118,7 +118,7 @@ export default class NotificationsService {
   }
 
   /**
-   * Sends the design review slack notifications for all design reviews scheduled for today
+   * Sends Slack notifications for all events scheduled for today whose event type has sendSlackNotifications enabled
    */
   static async sendEventSlackNotifications() {
     const endOfToday = startOfDayTomorrow();
@@ -142,6 +142,8 @@ export default class NotificationsService {
         optionalMembers: { include: { userSettings: true } },
         userCreated: { include: { userSettings: true } },
         scheduledTimes: true,
+        teams: true,
+        eventType: true,
         workPackages: {
           include: {
             wbsElement: true,
@@ -156,12 +158,18 @@ export default class NotificationsService {
       }
     });
 
-    const desginReviewEventTeamMap = new Map<string, EventWithAttendees[]>();
+    const eventTeamMap = new Map<string, EventWithAttendees[]>();
 
     events.forEach((event) => {
-      // Get all unique teams from all work packages associated with this event
+      // Collect unique team Slack IDs: first from teams directly on the event, then from work packages
       const teamSlackIds = new Set<string>();
 
+      event.teams.forEach((team) => {
+        if (team.slackId) {
+          teamSlackIds.add(team.slackId);
+        }
+      });
+
       event.workPackages.forEach((workPackage) => {
         workPackage.project.teams.forEach((team) => {
           if (team.slackId) {
@@ -171,7 +179,7 @@ export default class NotificationsService {
       });
 
       teamSlackIds.forEach((teamSlackId) => {
-        const currentEvents = desginReviewEventTeamMap.get(teamSlackId);
+        const currentEvents = eventTeamMap.get(teamSlackId);
         const eventWithAttendees = {
           ...event,
           attendees: event.requiredMembers.concat(event.optionalMembers).concat(event.userCreated),
@@ -181,20 +189,20 @@ export default class NotificationsService {
         if (currentEvents) {
           currentEvents.push(eventWithAttendees);
         } else {
-          desginReviewEventTeamMap.set(teamSlackId, [eventWithAttendees]);
+          eventTeamMap.set(teamSlackId, [eventWithAttendees]);
         }
       });
     });
 
-    // Send the notifications to each team for their respective design reviews
-    const promises = Array.from(desginReviewEventTeamMap).map(async ([slackId, events]) => {
+    // Send the notifications to each team for their respective events
+    const promises = Array.from(eventTeamMap).map(async ([slackId, events]) => {
       const messageBlock = events
         .map((event) => {
           const zoomLink = event.zoomLink ? `<${event.zoomLink}|Zoom Link>\n` : '';
           const questionDocLink = event.questionDocumentLink ? `<${event.questionDocumentLink}|Question Doc Link>\n` : '';
 
-          // Get work package names for this event
           const workPackageNames = event.workPackages.map((wp) => wp.wbsElement.name).join(', ');
+          const workPackagesPart = workPackageNames ? ` (${workPackageNames})` : '';
 
           // Get the earliest scheduled start time for display
           const [earliestSlot] = event.scheduledTimes
@@ -203,17 +211,17 @@ export default class NotificationsService {
           const timeDisplay = earliestSlot ? formatTimeForSlack(new Date(earliestSlot.startTime!)) : 'TBD';
 
           return (
-            `${usersToSlackPings(event.attendees ?? [])} ${event.title} (${workPackageNames}) ` +
+            `${usersToSlackPings(event.attendees ?? [])} *${event.eventType.name}*: ${event.title}${workPackagesPart} ` +
             `will be having an event today at ${timeDisplay} ET! ` +
             zoomLink +
             questionDocLink
           );
         })
         .join('\n\n');
 
-      // messageBlock will be empty if there are design reviews with no attendees
+      // messageBlock will be empty if there are events with no attendees
       if (messageBlock !== '')
-        await sendMessage(slackId, ':calendar: :clock9: Upcoming Design Reviews! :clock9: :calendar: \n\n\n' + messageBlock);
+        await sendMessage(slackId, ':calendar: :clock9: Upcoming Events! :clock9: :calendar: \n\n\n' + messageBlock);
     });
 
     await Promise.all(promises);

src/backend/src/utils/notifications.utils.ts

@@ -1,4 +1,4 @@
-import { Task as Prisma_Task, WBS_Element, Event, Work_Package } from '@prisma/client';
+import { Task as Prisma_Task, WBS_Element, Event, Work_Package, Team, Event_Type } from '@prisma/client';
 import { UserWithSettings } from './auth.utils.js';
 import { ScheduleSlot } from 'shared';
 
@@ -10,14 +10,16 @@ export type TaskWithAssignees = Prisma_Task & {
 export type EventWithAttendees = Event & {
   attendees: UserWithSettings[];
   scheduledTimes: ScheduleSlot[];
+  teams: Team[];
+  eventType: Event_Type;
   workPackages: (Work_Package & {
     wbsElement: WBS_Element;
   })[];
 };
 
 export const usersToSlackPings = (users: UserWithSettings[]) => {
   // https://api.slack.com/reference/surfaces/formatting#mentioning-users
-  return users.map(userToSlackPing).join(' ');
+  return users.filter((user) => user.userSettings?.slackId).map(userToSlackPing).join(' ');
 };
 
 export const userToSlackPing = (user: UserWithSettings) => {

src/frontend/src/pages/AdminToolsPage/ScheduleConfig/EventType/EventTypeFormModal.tsx

@@ -1,11 +1,11 @@
-import { Box, FormHelperText, Typography, Checkbox, FormControl, Select, MenuItem } from '@mui/material';
+import { Box, FormHelperText, Typography, Checkbox, FormControl, Select, MenuItem, Tooltip } from '@mui/material';
 import NERFormModal from '../../../../components/NERFormModal';
 import ReactHookTextField from '../../../../components/ReactHookTextField';
 import { useToast } from '../../../../hooks/toasts.hooks';
 import { useForm, Controller } from 'react-hook-form';
 import * as yup from 'yup';
 import { yupResolver } from '@hookform/resolvers/yup';
-import React from 'react';
+import React, { useEffect } from 'react';
 import { EventType } from 'shared';
 import useFormPersist from 'react-hook-form-persist';
 import { FormStorageKey } from '../../../../utils/form';
@@ -116,6 +116,16 @@ export const EventTypeFormModal: React.FC<EventTypeFormModalProps> = ({ open, on
     setValue
   });
 
+  const watchTeams = watch('teams');
+  const watchWorkPackage = watch('workPackage');
+  const notificationsDisabled = !watchTeams && !watchWorkPackage;
+
+  useEffect(() => {
+    if (notificationsDisabled) {
+      setValue('sendSlackNotifications', false);
+    }
+  }, [notificationsDisabled, setValue]);
+
   const onFormSubmit = async (data: EventTypeFormValues) => {
     try {
       await onSubmit(data);
@@ -829,12 +839,22 @@ export const EventTypeFormModal: React.FC<EventTypeFormModalProps> = ({ open, on
             control={control}
             name="sendSlackNotifications"
             render={({ field: { onChange, value } }) => (
-              <Checkbox checked={value} onChange={onChange} sx={{ color: 'white', '&.Mui-checked': { color: 'white' } }} />
+              <Checkbox
+                checked={value}
+                onChange={onChange}
+                disabled={notificationsDisabled}
+                sx={{ color: 'white', '&.Mui-checked': { color: 'white' } }}
+              />
             )}
           />
-          <NotificationsIcon sx={{ color: 'white', mr: 1 }} />
+          <Tooltip
+            title={notificationsDisabled ? 'Slack notifications cannot be sent without teams or work packages enabled' : ''}
+            placement="right"
+          >
+            <NotificationsIcon sx={{ color: notificationsDisabled ? 'rgba(255,255,255,0.3)' : 'white', mr: 1 }} />
+          </Tooltip>
           <Box sx={{ flex: 1 }}>
-            <Typography variant="body2" sx={{ color: 'white' }}>
+            <Typography variant="body2" sx={{ color: notificationsDisabled ? 'rgba(255,255,255,0.3)' : 'white' }}>
               Send Slack Notifications
             </Typography>
           </Box>

src/frontend/src/pages/CalendarPage/Components/EventModal.tsx

@@ -55,7 +55,6 @@ import Tooltip from '@mui/material/Tooltip';
 import { convertDayToInt, convertIntToDay } from '../../../utils/calendar.utils';
 import EditSeriesConfirmationModal from './EditSeriesConfirmationModal';
 import NotificationsIcon from '@mui/icons-material/Notifications';
-import NERSwitch from '../../../components/NERSwitch';
 import WarningAmberIcon from '@mui/icons-material/WarningAmber';
 
 export interface EventFormValues {