Skip to content
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

Attention Message in ReactPageScroller Component #46

Open
lagutinv opened this issue Apr 25, 2020 · 1 comment
Open

Attention Message in ReactPageScroller Component #46

lagutinv opened this issue Apr 25, 2020 · 1 comment

Comments

@lagutinv
Copy link

Hello!

I got next message in ReactPageScroller Component

Line 227:21: The ref value 'pageContainer.current' will likely have changed by the time this effect cleanup function runs. If this ref points to a node rendered by React, copy 'pageContainer.current' to a variable inside the effect, and use that variable in the cleanup function react-hooks/exhaustive-deps
Line 238:6: React Hook useEffect has a missing dependency: 'checkRenderOnMount'. Either include it or remove the dependency array react-hooks/exhaustive-deps
Line 294:6: React Hook useEffect has missing dependencies: 'animationTimer', 'animationTimerBuffer', 'children', 'componentIndex', and 'componentsToRenderLength'. Either include them or remove the dependency array react-hooks/exhaustive-deps

Is it possible to fix it somehow?

Thank you very much

@VikLiegostaiev
Copy link
Owner

Hi @lagutinv , thanks for this information, will fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants