Fix other type issues

theopensystemslab · Aug 29, 2024 · 287f3ba · 287f3ba
1 parent 86dbdf4
commit 287f3ba
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 3 deletions.
diff --git a/editor.planx.uk/src/components/Toast/types.ts b/editor.planx.uk/src/components/Toast/types.ts
@@ -20,3 +20,12 @@ type DeleteToast = {
   type: "DELETE_TOAST";
   payload: { id: number };
 };
+
+export type ToastContextType = {
+  addToast: (type: ToastType, message: string) => void;
+  remove: (id: number) => void;
+  success: (message: string) => void;
+  warning: (message: string) => void;
+  info: (message: string) => void;
+  error: (message: string) => void;
+} | null;
diff --git a/editor.planx.uk/src/contexts/ToastContext.tsx b/editor.planx.uk/src/contexts/ToastContext.tsx
@@ -1,9 +1,13 @@
 import ToastContainer from "components/Toast/ToastContainer";
-import { ToastState, ToastType } from "components/Toast/types";
+import {
+  ToastContextType,
+  ToastState,
+  ToastType,
+} from "components/Toast/types";
 import React, { createContext, ReactNode, useReducer } from "react";
 import { toastReducer } from "reducers/toastReducer";
 
-export const ToastContext = createContext(null);
+export const ToastContext = createContext<ToastContextType>(null);
 
 const initialState: ToastState = {
   toasts: [],
@@ -36,7 +40,14 @@ export const ToastContextProvider = ({
     addToast("error", message);
   };
 
-  const value = { success, warning, info, error, remove };
+  const value: ToastContextType = {
+    success,
+    warning,
+    info,
+    error,
+    remove,
+    addToast,
+  };
 
   return (
     <ToastContext.Provider value={value}>

diff --git a/editor.planx.uk/src/reducers/toastReducer.ts b/editor.planx.uk/src/reducers/toastReducer.ts
@@ -17,6 +17,8 @@ export const toastReducer = (state: ToastState, action: ToastAction) => {
       };
     }
     default:
+      // @ts-ignore
+      //   Typescript complains because action is of type 'never' here
       throw new Error(`Unhandled action type: ${action.type}`);
   }
 };