support custom serialization/deserialization. fixes #75

[jedelson-pagerduty]

The solution described in #75 (comment) doesn't work (at least for my use case) because setItem accepts a string whereas I (and I suspect @tpict) need to control how stringify/parse actually work.

[polemius]

Could you please update documentation as well?

[jedelson-pagerduty]

Could you please update documentation as well?

Done

[polemius]

Please add mention of converter to Usage section

[jedelson-pagerduty]

Please add mention of converter to Usage section

I assume in the Usage section it makes sense to show the default but please let me know if you think something else should go there.

[polemius]

I assume in the Usage section it makes sense to show the default but please let me know if you think something else should go there.

I thought maybe we could show in Usage section all options like:

import { recoilPersist } from 'recoil-persist'

const { persistAtom } = recoilPersist({
  key: 'recoil-persist', // this key is using to store data in local storage
  storage: localStorage, // configurate which storage will be used to store the data
  converter: JSON, // .... 
})

[jedelson-pagerduty]

Yeah, what I put there is

import { recoilPersist } from 'recoil-persist'
const { persistAtom } = recoilPersist({
  key: 'recoil-persist', // this key is using to store data in local storage
  storage: localStorage, // configurate which storage will be used to store the data
  converter: { // configurate how values will be serialized/deserialized in storage
    parse: (value) => JSON.parse,
    stringify: (value) =>  JSON.stringify
  }
})

Although converter: JSON would be semantically the same thing, I think it's useful to show both functions. WDYT?

[polemius]

converter: { // configurate how values will be serialized/deserialized in storage
    parse: (value) => JSON.parse(value),
    stringify: (value) =>  JSON.stringify(value)
}

you need to pass values to function

[polemius]

Good job! Thank you!

[polemius]

Released in 5.0.0 🚀