Redux: Rename createStore() second argument from initialState to preloadedState
It has caused enough confusion. This argument is only meant for providing persisted state, not for putting an object literal with all app鈥檚 initial state in one place. We should make it more clear by renaming the argument both in the source and everywhere in the docs.
gaearon
All 8 comments
(Arguably it鈥檚 also used for server rendering. Alternative naming suggestions are welcome. I just want to make it clear that it鈥檚 supposed to be generated and not written by hand.)
gaearon
on 18 May 2016
importedState? preloadedState?
markerikson
on 18 May 2016
preloadedState looks better. More ideas are welcome.
gaearon
on 18 May 2016
defaultState
BerkeleyTrue
on 18 May 2016
preloadedState looks good. Then how about stateToSync?
JisuPark
on 18 May 2016
My only argument to keep it as initialState is when referring to "currentState" it sounds more fluid to say "initial" -> "current" as opposed to "preloaded" -> "current".
DavidBrear
on 19 May 2016
W00t! Actually rather amused that my offhand suggestion of "preloadedState" worked out :)
markerikson
on 27 May 2016
:-)
gaearon
on 27 May 2016
Related issues
olalonde
路
3Comments
ms88privat
路
3Comments
elado
路
3Comments
vslinko
路
3Comments
parallelthought
路
3Comments
Most helpful comment
My only argument to keep it as initialState is when referring to "currentState" it sounds more fluid to say "initial" -> "current" as opposed to "preloaded" -> "current".