Warn using consola instead of throwing Error

[aldarund]

No description provided.

[TheAlexLichter]

We might want to consider stringify Symbols (otherwise apollo breaks as described here)

[aldarund]

@manniL do u know a good way to serialize them? I dont know :)
And with this changes it will print warning instead of breaking. They wasnt serializing before too.

[TheAlexLichter]

The change itself is fine 👏

Not sure. We should consult the serialize-javascript source code for this I think 🤔

[aldarund]

seializa-javascript dont do anything with symbols
https://runkit.com/5baa3cac92a46f00121e2985/5bb4e0d021ea590012334d7d

[TheAlexLichter]

Ah, well then we should just preserve them as they do I guess 🤷‍♂️

[clarkdo]

LGTM

[victor-perez]

This will not fix Rich-Harris#15

[aldarund]

I dont think vuex is supposed to be work with immutable.js in first place.
https://forum.vuejs.org/t/immutable-js-with-vue/6366/8
vuejs/vuex#802 (comment)

[victor-perez]

@aldarund before the switch to devalue there was no problem with using immutable this update is breaking this. And you can use immutable easily with vuex even with Nuxt using https://www.npmjs.com/package/vuex-immutable-replace-state
But that is not my point, devalue state that is like JSON.stringify but it doesn't handle Object that have a toJSON method.

[aldarund]

I added support for object with toJSON method