Fix: Added trycatch() to handleWeeklyPickChange(), added a throw erro…

…r inside onWeeklyPickChange(), and updated IData to IWeekData
LetsGetTechnical · Jul 26, 2024 · 4933a40 · 4933a40
1 parent baa8836
commit 4933a40
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/app/(main)/league/[leagueId]/entry/[entryId]/week/Week.interface.ts b/app/(main)/league/[leagueId]/entry/[entryId]/week/Week.interface.ts
@@ -9,7 +9,7 @@ import {
   IUser,
 } from '@/api/apiFunctions.interface';
 
-export interface IData {
+export interface IWeekData {
   target: {
     value: string;
   };
@@ -24,7 +24,7 @@ export interface IWeekParams {
 }
 
 export interface IWeeklyPickChange {
-  data: IData;
+  data: IWeekData;
   entry: string;
   league: string;
   NFLTeams: INFLTeam[];

diff --git a/app/(main)/league/[leagueId]/entry/[entryId]/week/Week.tsx b/app/(main)/league/[leagueId]/entry/[entryId]/week/Week.tsx
@@ -101,7 +101,11 @@ const Week = ({ entry, league, NFLTeams, week }: IWeekProps): JSX.Element => {
       weeklyPicks,
       week,
     };
-    await onWeeklyPickChange(params);
+    try {
+      await onWeeklyPickChange(params);
+    } catch (error) {
+      console.error('Error while changing pick:', error);
+    }
   };
 
   useEffect(() => {

diff --git a/app/(main)/league/[leagueId]/entry/[entryId]/week/WeekHelper.tsx b/app/(main)/league/[leagueId]/entry/[entryId]/week/WeekHelper.tsx
@@ -80,12 +80,12 @@ export const onWeeklyPickChange = async ({
       />,
     );
   } catch (error) {
-    console.error('Submission error:', error);
     toast.custom(
       <Alert
         variant={AlertVariants.Error}
         message="There was an error processing your request."
       />,
     );
+    throw new Error(`Error running onWeeklyPickChange: ${error}`);
   }
 };