search

dustingetz / react-cursor

Immutable state for React.js
1.03k stars 50 forks source link

Add Cursors support for dissoc #94

Closed danielmiladinov closed 8 years ago

dustingetz commented 8 years ago

Thanks this is great. It also needs to work for maps - but that can be done in a separate pass. Do you want to take a whack at that or should I just merge this?

dustingetz commented 8 years ago

I'm also thinking the remove fn should be pushed into the update-in dependency. And that remove fn should work on key or index. But let me know if I should just merge this for now.

danielmiladinov commented 8 years ago

I can do both, in this PR.

danielmiladinov commented 8 years ago

Actually, moving the remove fn to update-in will be a separate pr to update-in