Skip to content

Commit

Permalink
added dependencies to the useEffect hook
Browse files Browse the repository at this point in the history
  • Loading branch information
amolsasane committed Jul 5, 2024
1 parent 20bb5f6 commit 1b0af25
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion src/hooks/useMovieTrailer.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ const useMovieTrailer = (id) => {

useEffect(() => {
getMovieVideos();
}, [id]);
}, [dispatch, id]);
};

export default useMovieTrailer;
2 changes: 1 addition & 1 deletion src/hooks/useNowPlayingMovies.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ const useNowPlayingMovies = () => {

useEffect(() => {
getNowPlayingMovies();
}, []);
}, [dispatch, getNowPlayingMovies]);
};

export default useNowPlayingMovies;
2 changes: 1 addition & 1 deletion src/hooks/usePopularMovies.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ const usePopularMovies = () => {

useEffect(() => {
getPopularMovies();
}, []);
}, [dispatch, getPopularMovies]);
};

export default usePopularMovies;
2 changes: 1 addition & 1 deletion src/hooks/useTopRatedMovies.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ const useTopRateMovies = () => {

useEffect(() => {
getTopRatedMovies();
}, []);
}, [dispatch, getTopRatedMovies]);
};

export default useTopRateMovies;
2 changes: 1 addition & 1 deletion src/hooks/useUpcomingMovies.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ const useUpcomingMovies = () => {

useEffect(() => {
getUpcomingMovies();
}, []);
}, [dispatch, getUpcomingMovies]);
};

export default useUpcomingMovies;

0 comments on commit 1b0af25

Please sign in to comment.