-
Notifications
You must be signed in to change notification settings - Fork 307
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
Listener is missing to register/emit events in some cases. #71
Comments
Thanks for the report, we'll look into it! I also noticed an issue with hot module reload, some listners are not setup properly in the provider |
Got the same issue when trying to register multiple Preview windows. |
I also found a scenario in which the listeners give an error. Will have to debug this inside the context. https://codesandbox.io/s/jolly-poincare-zjoxf?file=/src/App.tsx (any change that triggers a hot reload throws an error) |
It was quite tricky to point down exactly when it's happening. But at times, the
listener
fromuseSandpack
is failing to listen for events emitted by the sandpack bundler.https://codesandbox.io/s/eloquent-mccarthy-qr09r?file=/src/App.tsx
STR
csb
preview. And in some instances the listener won't log the events.Screen.Recording.2021-06-29.at.8.16.18.AM.mov
The text was updated successfully, but these errors were encountered: