Questions about serialization / deserialization about flummox HOT 3 CLOSED

Hi! Thanks for the feedback.

I agree that this seems redundant if your stores only contain JSON stringifiable (is that a word?) data, but it's often the case that Store data is not JSON stringifiable — for instance, using Immutable.js data types. In that case, JSON.stringify() and JSON.parse() will not work, hence the current solution, which is just to let the user define how to serialize and deserialize state.

The calls to JSON.stringify() and JSON.parse() you've identified in the code are only for the top-level tree of store keys to store data strings. So the parsed object looks like this:

{
  storeA: '{"foo": "bar"}', // String of JSON stringified data, but it could actually be anything returned by `Store#serialize()`
  storeB: '{"foo": "bar"}'
}

Then we iterate over the values and send the strings to the stores for deserialization. In other words, Flux#deserialize makes no assumptions about the type of data in your stores.

Does that make sense? I'm open to suggestions for how to make this better.

from flummox.

Ah! I read your comment again, and looked at the code again, and now I see the real issue — I'm passing the entire state string to Store#deserialize, not the store string! Oops! I'll fix this right away.

My previous comment is how it should work, though, so if you have any suggestions for how it could be better I'd still like to hear them :)

from flummox.

Fixed in 2.5.1

from flummox.