-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Missing Dependencies of useEffect Hooks #69
Comments
Hi @shubhanshu02 bhaiya, is this issue open for now . |
Yes, @Devansh-gaur-1611. You are welcome to put up a PR for that. |
Thanks, @shubhanshu02 bhaiya for assigning me. I have started working on this issue and will report you about progress in some time. |
Hello @shubhanshu02 bhaiya, I have checked the code and find out that all the dependencies which are specified above have already been added. So bhaiya this issue can be closed. |
Oh, yes right. These changes are covered in #131. Thanks for the catch @Devansh-gaur-1611. |
Description
In some files, the
useEffect
has missing dependencies. The issue proposes including such unmet dependencies in the dependencies array of theuseEffect
hook.Occurrences
File:
pages/hackathon/[slug]/teams/[team_id]/submit.js
Line: 57
Missing Dependency:
token
File:
pages/hackathon/[slug]/index.js
Line: 77
Missing Dependency:
hackathon
The text was updated successfully, but these errors were encountered: