Skip to content
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

Only Propagate Changed Properties #16

Closed
curran opened this issue Aug 28, 2018 · 0 comments
Closed

Only Propagate Changed Properties #16

curran opened this issue Aug 28, 2018 · 0 comments

Comments

@curran
Copy link
Member

curran commented Aug 28, 2018

The .set method marks all properties passed in as changed, whereas they may not have actually changed.

When used in Redux-style programs, it makes sense to simply pass in all properties. It would be great to be able to use Topologica in this way, and have it "deal with" checking which properties are different than they were before, and only propagating the ones that changed.

The check could be strict equality ===. This would fail in cases where mutated objects are passed in, but I think it's fine to not support this, and say the library only works with immutable property values.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant