gentleman-programming / front-end-clean-architecture Goto Github PK

Alan buenas tarde, estoy aprendiendo y mirando el tema de hook y cuando implemento algo de este estilo el linter me sale con el siguiente mensaje: React Hook useEffect has a missing dependency: 'cancelEndpoint'. Either include it or remove the dependency array.eslint react-hooks/exhaustive-deps

https://github.com/AppleLAN/CustomhooksClass/blob/434c34da6a62d67c14df82dbd740fddffdb0df49/src/hooks/useFetchAndLoad.ts#L32

para corregir el warning hice lo siguiente:

useEffect(() => {
    return () => {
      const cancelEndpoint = () => {
        setLoading(false);
        console.log("[useFetchAndLoad] cancelEndpoint");
        controller && controller.abort();
      };
    
      cancelEndpoint();
    };
  }, [controller]);

Esto es correcto o no es necesario realizar esta modificación?

en otros casos me indica que si lo tengo fuera del useEffect use un useCallback pero no esto entendiendo del todo como solucionar correctamente esto del array de dependencias.

**The 'searchEmployee' function makes the dependencies of useEffect Hook (at line 64) change on every render. Move it inside the useEffect callback. Alternatively, wrap the definition of 'searchEmployee' in its own useCallback() Hook.eslint **react-hooks/exhaustive-deps