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’s 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’s also used for server rendering. Alternative naming suggestions are welcome. I just want to make it clear that it’s 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
caojinli
·
3Comments
timdorr
·
3Comments
ilearnio
·
3Comments
dmitry-zaets
·
3Comments
cloudfroster
·
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".