We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
store.getState()
In v7, we definitely bailed out of useSelector if the store state hadn't changed:
useSelector
react-redux/src/hooks/useSelector.js
Lines 67 to 71 in 49f7680
Per reduxjs/redux#4627 , there's questions of whether it still does that correctly in v8 with useSyncExternalStore. Should look into this.
useSyncExternalStore
(also, there's a question of whether we should bail out inside subscriber callbacks, or before calling them)
The text was updated successfully, but these errors were encountered:
Hey @markerikson, I fixed it in react.
Sorry, something went wrong.
No branches or pull requests
In v7, we definitely bailed out of
useSelector
if the store state hadn't changed:react-redux/src/hooks/useSelector.js
Lines 67 to 71 in 49f7680
Per reduxjs/redux#4627 , there's questions of whether it still does that correctly in v8 with
useSyncExternalStore
. Should look into this.(also, there's a question of whether we should bail out inside subscriber callbacks, or before calling them)
The text was updated successfully, but these errors were encountered: