-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
Warning: Maximum update depth exceeded. #281
Comments
I reopen the issue is happening |
@elenitaex5 Can you create a demo snack? https://snack.expo.dev/ |
I ran into the same issue. I was not able to create a repro yet and it's late but I narrowed down the problem in my application: If
I'll try to figure out what exactly is causing the render loop tomorrow. An obvious workaround is to wrap the function passed to |
I'm creating a snack but there's a problem with a dependency in the snack. I include it as soon as I have it. |
I won't be able to create a Snack for you because reproducing this issue in Snack causes the browser to crash due to how Snack works 😅 Here's a repro of the issue, or at least how the issue was triggered in my case: How does it happen? There are two crucial issues that both need to happen at the same time:
In my case the bug was in my own
I'm not entirely sure if this should be considered a bug in the library or a bug in the custom code and if a fix or a warning and better documentation is required. I already spent few hours on debugging it and ran out of allotted time to continue so I didn't check if the For now the workaround is to make sure |
Same for me.
|
Hi! I found your library and seems to works well, but there is an issue in react-native-web (running with EXPO and webpack)
As soon as
OutsidePressHandler
is added it starts warning like below:Warning: Maximum update depth exceeded. This can happen when a component calls setState inside useEffect, but useEffect either doesn't have a dependency array, or one of the dependencies changes on every render.
functionality is ok, but I thing there something missing in the library.
The text was updated successfully, but these errors were encountered: