edy / redux-persist-transform-filter

Filter transformator for redux-persist
MIT License
190 stars 21 forks source link

Added support for immutablejs #5

Closed actra-gschuster closed 7 years ago

rt2zz commented 7 years ago

@actra-gschuster thanks for contributing, however we cannot reasonably expect all users to have or add immutablejs. It looks like the only way to reasonably achieve this is to make a separate transform with immutable support. Following the naming convention of the ecosystem you might call the modules something like redux-persist-transform-filter-immutable. Long but explicit :)

actra-gschuster commented 7 years ago

Using immutablejs is optional, it still supports plain objects. But nevermind, I'll set up a repo and publish it there.

rt2zz commented 7 years ago

immutable objects is optional, but with this PR having the dependency is required

edy commented 7 years ago

closed. see above comments and https://github.com/actra-development/redux-persist-transform-filter-immutable