search

klarna / electron-redux

Use redux in the main and browser processes in electron
MIT License
743 stars 94 forks source link

Make sure the store serialization follows redux recommendations #261

Closed matmalkowski closed 3 years ago

matmalkowski commented 3 years ago

Actually I don't like idea of having this by default baked into the lib - it goes kinda against the redux recommendations itself:

It is highly recommended that you only put plain serializable objects, arrays, and primitives into your store.

I would like to extract this and make it configurable so users when using the middleware can configure it with their own serializer at their own risk (and we ofc can provide them with example serializer used here 👍🏻 )

As a follow up we should address this on v2

matmalkowski commented 3 years ago

I'm going to work on this

matmalkowski commented 3 years ago

:tada: This issue has been resolved in version 2.0.0-alpha.2 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

matmalkowski commented 3 years ago

resolved #272