tested and working slack bot

Author: chpy04

src/backend/src/services/part-review.services.ts

@@ -11,9 +11,7 @@ import {
   isAtLeastRank,
   RoleEnum,
   Review_Status,
-  validateWBS,
-  wbsNumComparator,
-  wbsNamePipe
+  validateWBS
 } from 'shared';
 import {
   AccessDeniedAdminOnlyException,
@@ -46,7 +44,6 @@ import { uploadFile, downloadFile } from '../utils/google-integration.utils';
 import ProjectsService from './projects.services';
 import { sendPartAssignmentPopUp, sendPartReviewRequestPopUp } from '../utils/pop-up.utils';
 import { sendSlackPartAssignmentNotif, sendSlackPartReviewRequestNotif } from '../utils/slack.utils';
-import { get } from 'http';
 import { getUserWithSettingsQueryArgs } from '../prisma-query-args/user.query-args';
 
 export default class PartReviewService {
@@ -924,9 +921,9 @@ export default class PartReviewService {
     if (reviewer.userSettings?.slackId) {
       await sendSlackPartReviewRequestNotif(
         reviewer.userSettings.slackId,
-        partLink,
+        part.project.wbsElement.name,
         part.commonName,
-        part.project.wbsElement.name
+        partLink
       );
     }
   }
@@ -971,9 +968,9 @@ export default class PartReviewService {
     if (assignee.userSettings?.slackId) {
       await sendSlackPartAssignmentNotif(
         assignee.userSettings.slackId,
-        partLink,
+        part.project.wbsElement.name,
         part.commonName,
-        part.project.wbsElement.name
+        partLink
       );
     }
   }

src/backend/src/utils/slack.utils.ts

@@ -561,7 +561,7 @@ export const sendSlackPartReviewRequestNotif = async (
 ) => {
   if (process.env.NODE_ENV !== 'production') return; // don't send msgs unless in prod
 
-  const msg = `Your review has been requested on part: ${partName} for project ${projectName}`;
+  const msg = `Your review has been requested on part: ${partName} for project: ${projectName}`;
   const link = `https://finishlinebyner.com${partLink}`;
   const linkButtonText = 'View Part';
   await sendMessage(slackId, msg, link, linkButtonText);
@@ -575,7 +575,7 @@ export const sendSlackPartAssignmentNotif = async (
 ) => {
   if (process.env.NODE_ENV !== 'production') return; // don't send msgs unless in prod
 
-  const msg = `You have been assigned to part: ${partName} on project ${projectName}`;
+  const msg = `You have been assigned to part: ${partName} on project: ${projectName}`;
   const link = `https://finishlinebyner.com${partLink}`;
   const linkButtonText = 'View Part';
   await sendMessage(slackId, msg, link, linkButtonText);

src/frontend/src/pages/PartPage/PartPageComponents/PartOverview.tsx

@@ -5,6 +5,7 @@ import { Part, Review_Status, User } from 'shared';
 import { getReviewStatusDisplayName, getStatusColor } from '../../../utils/part.utils';
 import { useNotifyPartAssignee, useNotifyPartReviewer } from '../../../hooks/part-review.hooks';
 import { useState } from 'react';
+import { useToast } from '../../../hooks/toasts.hooks';
 
 const PartReviewStatusPill = (status: Review_Status) => {
   return (
@@ -46,8 +47,9 @@ interface PartPageOverviewProps {
 }
 
 const PartOverview: React.FC<PartPageOverviewProps> = ({ part }: PartPageOverviewProps) => {
-  const { mutate: notifyPartAssignee } = useNotifyPartAssignee();
-  const { mutate: notifyPartReviewer } = useNotifyPartReviewer();
+  const { mutateAsync: notifyPartAssignee } = useNotifyPartAssignee();
+  const { mutateAsync: notifyPartReviewer } = useNotifyPartReviewer();
+  const toast = useToast();
 
   const [notifiedUserIds, setNotifiedUserIds] = useState<Set<string>>(new Set());
 
@@ -60,13 +62,20 @@ const PartOverview: React.FC<PartPageOverviewProps> = ({ part }: PartPageOvervie
         {status !== Review_Status.APPROVED && (
           <IconButton
             size="small"
-            onClick={() => {
-              if (isReviewer) {
-                notifyPartReviewer({ partId, reviewerId: anyUser.userId });
-              } else {
-                notifyPartAssignee({ partId, assigneeId: anyUser.userId });
+            onClick={async () => {
+              try {
+                if (isReviewer) {
+                  await notifyPartReviewer({ partId, reviewerId: anyUser.userId });
+                } else {
+                  await notifyPartAssignee({ partId, assigneeId: anyUser.userId });
+                }
+                setNotifiedUserIds((ids) => ids.add(anyUser.userId));
+                toast.success('' + (isReviewer ? 'Reviewer' : 'Assignee') + ' notified');
+              } catch (error) {
+                if (error instanceof Error) {
+                  toast.error(error.message);
+                }
               }
-              setNotifiedUserIds((ids) => ids.add(anyUser.userId));
             }}
             sx={{
               backgroundColor: '#444444',