-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support non-JSON values #60
Comments
I guess I think we better add that in |
Oh, that seems better, indeed. |
@zalmoxisus @alexkuz I wrote a helper function to do this, let me know if it solves your use case |
@zalmoxisus thank you! |
Should work well now. Thanks to everybody for working on this! |
The extension serializes state with
jsan
, losing values likeundefined
orfunction
.The way to fix this is to use
node-stringify
/eval
, instead ofjsan.stringify
/jsan.parse
.(there is probably a better option than
eval
, I'm not sure) :)The text was updated successfully, but these errors were encountered: