-
Notifications
You must be signed in to change notification settings - Fork 202
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
Doesn't work with class components wrapped in React.memo #50
Comments
It throws following exception:
The reason is that |
ok i might indeed fix it. and maybe even soon but React.memo is made for functional components: are you sure it even makes your class component pure? |
added a prove that this happens (a test that fails) to the following branch: Looks like a not too hard thing to fix, if anybody wants to try to do it. |
See #10 (comment)
The text was updated successfully, but these errors were encountered: