-
Notifications
You must be signed in to change notification settings - Fork 39
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
detecting state changes #33
Comments
OK I lied, this is not working. Is it by design that the stateChange function is called each time a state property is updated, and before return state is called? |
Ok I found a way to be able to group my state values, and still ahve the changes registered.
More hacky then before,... Is there another way? or are deep properties frowned upon? |
Ok Never mind.... The internet tells me to "keep your state flat". So I will trade _'s for .'s etc |
Change |
If I set a deep property like this this:
my state change listeners do not get called.
If I add a kind of cache buster at the root of my state object, then it works.
It seems to work and is pretty painless, but I thought I would just check to see if this was a bug, or if there is a more proper way to push changes so that they get detected.
The text was updated successfully, but these errors were encountered: