search

dustingetz / react-cursor

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

Add assertion preventing function references inside cursors #87

Open dustingetz opened 8 years ago

dustingetz commented 8 years ago

See https://github.com/dustingetz/react-cursor/pull/86 https://github.com/dustingetz/react-cursor/issues/19

dustingetz commented 8 years ago

Need to understand how to deal with class instances and objects with prototypes in cursor first

newyankeecodeshop commented 8 years ago

Disallowing the use of more complex types like Date, Map, Set might be problematic for cursor users down the road. I know there is something nice about having state that can be serialized to/from JSON or DOM storage, but not every component needs that. Something to consider.