It doesn't appear as though there's a simple way to know whether the store has been mounted from localStorage.

Would it make sense to add a reducer specific to the state of the adapter interaction which contained the current state: whether state has been loaded from the appropriate adapter, whether it has been initialized, if it is in the process of saving, etc.

I'd like to wait until the state is mounted from the adapter before initializing any components in react-native, as I'm storing the navigation state in localstorage.

Also wanted to know what you thought about adding a way to clear / reset the localstorage state on the top level store.

We get the following with the 1.0 breaking changes branch when firing up the server:

/Users/whatevs/projects/stuff/ui/node_modules/redux-localstorage/lib/persistState.js:29
var defaultStorage = (0, _adaptersLocalStorage2['default'])(localStorage);
                                                            ^
ReferenceError: localStorage is not defined
    at Object.<anonymous> (/Users/whatevs/projects/stuff/ui/node_modules/redux-localstorage/lib/persistState.js:29:61)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/Users/whatevs/projects/stuff/ui/node_modules/redux-localstorage/lib/index.js:17:23)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
 +407ms

I presume this is b/c of the server-side rendering. So, I added this to fix it in persistState.js:

if (typeof window === "undefined" ) {
  require('fs');
  var LocalStorage = require('node-localstorage').LocalStorage;
  localStorage = new LocalStorage('./scratch');
} else {
  localStorage = window.localStorage;
}
// this is existing 
var defaultStorage = (0, _adaptersLocalStorage2['default'])(localStorage);

It adds a dependency on node-localstorage, but that doesn't seem too unreasonable...does it?

Thoughts?

I've defined a slicer to specifiy a subset which is saved to localStorage. Inside one of my reducers I've got an initial state defined, comparable to this example.

const initialState = {
  list: [],
  favorites: [],
  isLoading: false
}

// the default value `initialState` won't be used anymore when there's 
// already a subset of the initialstate define in localstorage
export default function jobs (state = initialState, action) {
  const reduceFn = actionsMap[action.type]
  if (!reduceFn) return state
  return Object.assign({}, state, reduceFn(state, action))
}

My reduce won't consume the initial state anymore, since there's another initial state passed to this reducer, together with the action named "@@redux/INIT.

How should I merge both the initialState and the state coming from local storage?

A few thoughts:

1. I don't see any reason to use this library with only persistState but not mergePersistedState, so it doesn't make sense to make persistState as _the_ default export. Please consider making it work like this instead:

   import {persistState, mergePersistedState} from 'redux-localstorage';
   

2. It's not good to import something from a third-party library's inner directory structure because this creates an extra layer of code dependency. This will cause breaking changes when you simply wanna change your project's directory structure. For reference, React is also moving all the addons out of the main package into independent packages.

3. It feels really weird that in case I have to include a filter (which is gonna be very common for real world cases), my related code would go from simply:

import {compose, createStore} from 'redux';
import {persistState} from 'redux-localstorage';

compose(
  persistState()
)(createStore)

To this bloated mess:

import {compose, createStore} from 'redux';
import {persistState} from 'redux-localstorage';
import adapter from 'redux-localstorage/lib/adapters/localStorage';
import filter from 'redux-localstorage-filter';

compose(
  persistState(
    compose(
      filter('nested.key')
    )(adapter(window.localStorage))
  )
)(createStore)

The weirdest part is - I don't understand why we have to create the localStorage storage ourselves. Shouldn't this be handled by the library which is called precisely redux-localstorage?

If there is no justified benefit, couldn't we make it a bit simpler like this?

import {compose, createStore} from 'redux';
import {persistState, localStorage} from 'redux-localstorage';
import filter from 'redux-localstorage-filter';

compose(
  persistState(
    compose(
      filter('nested.key')
    )(localStorage)
  )
)(createStore)

Where the localStorage export is simply adapter(window.localStorage).

@elgerlambert Per the discussion in #13, it would be really nice if you could push the changes to npm.

Currently, I have to patch my version with the code from the PR.

redux-devtools currently uses persistState as a module, which conflicts with this module's persistState. Any way I can use both of these addons?

Thanks

I'm not sure how to setup the config parameter to properly use an Immutable object, This is how I configure the options:

import Immutable from 'immutable';

export const LocalStorageConfig = {
    serialize: (subset) => subset.toJson(),
    deserialize: (serializedData) => Immutable.fromJS(JSON.parse(serializedData)),
    merge: (initialState, persistedState) => initialState.mergeDeep(persistedState)
};

But I get an error with my selector, state.get is not a function :

export const menuSelector = createSelector(
    mainSelector,
        (state) => ({
            menu: state.get('menu')
    })
);

It looks like I'm missing something here.

Failed to retrieve initialize state from localStorage: ReferenceError: localStorage is not defined
    at /Users/azz0r/Sites/mobile-presales/node_modules/redux-localstorage/lib/persistState.js:59:38
    at /Users/azz0r/Sites/mobile-presales/node_modules/redux/lib/applyMiddleware.js:38:19
    at createStore (/Users/azz0r/Sites/mobile-presales/node_modules/redux/lib/createStore.js:59:33)

Any work arounds? This doesn't block redux-localstorage working but it leaves an initial error message in console.

Hi, I had a weird behaviour on a customer's mobile device : cookies where blocked in safari and the whole app wouldn't load and triggered this error from redux-localstorage: DOM Exception 18: An attempt was made to break through the security policy of the user agent. Seems to be related to accessing localstorage, I don't know if it is by design or if it should fail without breaking the app.

thanks, great work

Adds a local/session Storage that encodes the string before storing it and decodes it after getting it from the storage. Useful if you do not wish to store your state as plain text.

I'm using Babel 6.24, and after installing this package, it throws:

ERROR in ./~/redux-localstorage/lib/persistState.js
Module build failed: ReferenceError:
[BABEL] /home/emma/.../node_modules/redux-localstorage/lib/persistState.js:
Using removed Babel 5 option: /home/emma/.../node_modules/redux-localstorage/.babelrc.stage -
Check out the corresponding stage-x presets http://babeljs.io/docs/plugins/#presets

After removing node_modules/redux-localstorage/.babelrc, everything works fine.

Minor grammar fix

If you accept this PR, you might want to edit the repo's About description to match the change.

Comparing omission of parenthetical modifiers:

Before

• Store enhancer that syncs (a subset) of your Redux store state to localstorage.
• Store enhancer that syncs of your Redux store state to localstorage.

After

• Store enhancer that syncs (a subset of) your Redux store state to localStorage.
• Store enhancer that syncs your Redux store state to localStorage.

after I add @types/redux-localstorage, I could not use an adapter for a LocalForage type object.

import adapter from 'redux-localstorage/lib/adapters/localForage'; is now throwing an error:

TS2307: Cannot find module 'redux-localstorage/lib/adapters/localForage'.

However, if I import it this way: import adapter from 'redux-localstorage/lib/adapters/AsyncStorage';, the terminal says Module not found: Can't resolve 'redux-localstorage/lib/adapters/AsyncStorage' though it exists

can anyone help me with this issue? thanks!

I'm using redux-localstorage in Electron, and in the case the app crashes or someone uses Ctrl-C from the app started from the terminal, the latest changes to redux during the session will not be persisted.

What I would like to do is make sure it gets persisted from time to time, I'm not sure whether this is possible, it might be a more global issue with LocalStorage and how the application/browser exits, but I thought I'd ask if anyone has figured out a proper way of doing this.

Thanks

If I add new things to a reducer, if that reducer was persisted to local storage it's not being merged with the new things being added to the reducer.

Am I misunderstanding the merge property in the docs? Or is this not something this library is meant to do at all?

I've been trying to get this to work with composeWithDevTools from redux-devtools-extension like this

import {createStore,applyMiddleware,compose} from 'redux';
import { composeWithDevTools } from 'redux-devtools-extension';
import thunk from 'redux-thunk';
import persistState from 'redux-localstorage'

const store = createStore(
  reducers,
  composeWithDevTools(
    applyMiddleware(thunk),
    persistState(),
  )
)

export default store

However this breaks all the async requests in my app.

Any ideas?