karlhadwen / netflix

Subscribe to my YouTube channel: https://bit.ly/CognitiveSurge - Building Netflix Using React
MIT License
2.08k stars 577 forks source link

Search not working #12

Closed AbenezerBayeh closed 3 years ago

AbenezerBayeh commented 3 years ago

Hi Sir,

The search (which uses fuse.js) is not working and I'm getting this error:

webpackHotDevClient.js:138 ./src/hooks/use-auth-listener.js Line 21:6: React Hook useEffect has a missing dependency: 'firebase'. Either include it or remove the dependency array react-hooks/exhaustive-deps printWarnings @ webpackHotDevClient.js:138 handleWarnings @ webpackHotDevClient.js:143 push../node_modules/react-dev-utils/webpackHotDevClient.js.connection.onmessage @ webpackHotDevClient.js:210 webpackHotDevClient.js:138 ./src/hooks/use-content.js Line 24:6: React Hook useEffect has missing dependencies: 'firebase' and 'target'. Either include them or remove the dependency array react-hooks/exhaustive-deps printWarnings @ webpackHotDevClient.js:138 handleWarnings @ webpackHotDevClient.js:143 push../node_modules/react-dev-utils/webpackHotDevClient.js.connection.onmessage @ webpackHotDevClient.js:210 webpackHotDevClient.js:138 ./src/components/player/index.js Line 36:11: 'showPlayer' is assigned a value but never used no-unused-vars printWarnings @ webpackHotDevClient.js:138 handleWarnings @ webpackHotDevClient.js:143 push../node_modules/react-dev-utils/webpackHotDevClient.js.connection.onmessage @ webpackHotDevClient.js:210 webpackHotDevClient.js:138 ./src/containers/browse.js Line 39:6: React Hook useEffect has missing dependencies: 'category', 'slideRows', and 'slides'. Either include them or remove the dependency array. If 'setSlideRows' needs the current value of 'slides', you can also switch to useReducer instead of useState and read 'slides' in the reducer react-hooks/exhaustive-deps

Let me know if there's a quick fix for that! And great work on the website!

karlhadwen commented 3 years ago

Strange. Can you add the dependencies that React is talking about and see if there's any difference?

AbenezerBayeh commented 3 years ago

Thanks for the response Karl. I will make those changes and see if it works!