-
Notifications
You must be signed in to change notification settings - Fork 208
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
Seems to break React Server Components #364
Comments
Hi @Kyoss79! Can you point me where we are using React Context? It's been a while since I was in this codebase so I might be missing something but I can't find it. I wouldn't be surprised if react-waypoint is incompatible with newer React tech (like NextJS app dir), so one possible solution would be for you to move to IntersectionObserver (either via a library or directly against the browser API). |
By looking through the code, the problem seems to be within "react-is".
Mhmm |
I believe that code is part of the react repo: https://github.com/facebook/react/blob/1f5ce59dd7b6869b1a17ede65aa301002ef31d4b/packages/shared/ReactSymbols.js#L21-L24 |
i couldn't look any deeper into this and changed react-waypoint for react-intersection-observer. Kinda sad, I liked your API more. |
Hello,
been using this library up to now.
It seems the library is using React Context internally,
which breaks the new NextJS App Directory using Server Components.
The text was updated successfully, but these errors were encountered: