fix:fix eslint errors

mate-academy · Nov 15, 2024 · fa4c274 · fa4c274
1 parent 905c871
commit fa4c274
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 8 deletions.
diff --git a/src/hooks/useLocaLStorage.ts b/src/hooks/useLocaLStorage.ts
@@ -6,9 +6,11 @@ interface IUseLocalStorage {
 
 const useLocaLStorage = (key: string): IUseLocalStorage => {
   const setItem = (value: unknown): void => {
-    typeof value === 'string'
-      ? localStorage.setItem(key, value)
-      : localStorage.setItem(key, JSON.stringify(value));
+    if (typeof value === 'string') {
+      localStorage.setItem(key, value);
+    } else {
+      localStorage.setItem(key, JSON.stringify(value));
+    }
   };
 
   const getItem = (): unknown => {

diff --git a/src/hooks/useSelectedTodo.ts b/src/hooks/useSelectedTodo.ts
@@ -10,7 +10,7 @@ export const useSelectedTodo = () => {
         setSelectedTodo(null);
       }
     },
-    [setSelectedTodo],
+    [selectedTodo],
   );
 
   useLayoutEffect(() => {

diff --git a/src/hooks/useTodos.ts b/src/hooks/useTodos.ts
@@ -1,4 +1,4 @@
-import { useLayoutEffect, useState } from 'react';
+import { useCallback, useLayoutEffect, useState } from 'react';
 import { Todo } from '../types/Todo';
 import {
   getCompletedTodos,
@@ -13,11 +13,12 @@ export const useTodos = () => {
   const [todos, setTodos] = useState<Todo[]>([]);
   const { getItem, setItem } = useLocaLStorage('todos');
 
-  const fetchTodos = () => {
+  const fetchTodos = useCallback(() => {
     const localTodos = <Todo[]>getItem();
 
     setTodos(localTodos ? localTodos : []);
-  };
+  }, []);
+
   const addTodo = (title: string): Todo | void => {
     const todo = {
       title,
@@ -79,7 +80,7 @@ export const useTodos = () => {
 
   useLayoutEffect(() => {
     fetchTodos();
-  }, []);
+  }, [fetchTodos]);
 
   return {
     todos,