#1250: Added try/catch back to CheckList and ProposedSolutionsList

Author: sharonyang16

src/frontend/src/components/CheckList.tsx

@@ -14,6 +14,7 @@ import { Tooltip } from '@mui/material';
 import { User } from 'shared';
 import NERModal from './NERModal';
 import { fullNamePipe } from '../utils/pipes';
+import { useToast } from '../hooks/toasts.hooks';
 
 export type CheckListItem = {
   id: number;
@@ -34,6 +35,7 @@ const CheckList: React.FC<CheckListProps> = ({ title, items, isDisabled }) => {
   const { isLoading, mutateAsync } = useCheckDescriptionBullet();
   const [showConfirm, setShowConfirm] = useState<boolean>(false);
   const [currIdx, setCurrIdx] = useState<number>(-1);
+  const toast = useToast();
 
   const handleUncheck = async (idx: number) => {
     await handleCheck(idx);
@@ -44,7 +46,9 @@ const CheckList: React.FC<CheckListProps> = ({ title, items, isDisabled }) => {
     try {
       await mutateAsync({ userId: auth.user!.userId, descriptionId: items[idx].id });
     } catch (e) {
-      // do nothing; toasting error message will cause errors
+      if (e instanceof Error) {
+        toast.error(e.message);
+      }
     }
   };
 

src/frontend/src/pages/ChangeRequestDetailPage/ProposedSolutionsList.tsx

@@ -13,6 +13,7 @@ import ErrorPage from '../ErrorPage';
 import LoadingIndicator from '../../components/LoadingIndicator';
 import { useAuth } from '../../hooks/auth.hooks';
 import { Button } from '@mui/material';
+import { useToast } from '../../hooks/toasts.hooks';
 
 interface ProposedSolutionsListProps {
   proposedSolutions: ProposedSolution[];
@@ -24,6 +25,7 @@ const ProposedSolutionsList: React.FC<ProposedSolutionsListProps> = ({ proposedS
   const [showEditableForm, setShowEditableForm] = useState<boolean>(false);
   const auth = useAuth();
   const { isLoading, isError, error, mutateAsync } = useCreateProposeSolution();
+  const toast = useToast();
 
   if (isLoading || !auth.user) return <LoadingIndicator />;
   if (isError) return <ErrorPage message={error?.message} />;
@@ -44,7 +46,9 @@ const ProposedSolutionsList: React.FC<ProposedSolutionsListProps> = ({ proposedS
         budgetImpact
       });
     } catch (e) {
-      // do nothing; toasting error message will cause errors
+      if (e instanceof Error) {
+        toast.error(e.message);
+      }
     }
   };