Skip to content
This repository has been archived by the owner on Sep 11, 2018. It is now read-only.

Fixed issue 'Unexpected key counter found in previous state received … #1059

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

shendepu
Copy link

…by the reducer' when added window.__INITIAL_STATE of { counter: 1 }. The counter reducer is async injected, so there no reducer to key 'counter' when combineReducers in createStore intially. Therefore a warning from combineReducers shows in non-production mode, but the initial state of { counter: 1 } is removed in production mode

…by the reducer' when added window.__INITIAL_STATE of { counter: 1 }. The counter reducer is async injected, so there no reducer to key 'counter' when combineReducers in createStore intially. Therefore a warning from combineReducers shows in non-production mode, but the initial state of { counter: 1 } is removed in production mode
@codecov-io
Copy link

codecov-io commented Oct 24, 2016

Current coverage is 71.64% (diff: 66.66%)

Merging #1059 into master will decrease coverage by 0.93%

@@             master      #1059   diff @@
==========================================
  Files            12         12          
  Lines            62         67     +5   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits             45         48     +3   
- Misses           17         19     +2   
  Partials          0          0          

Powered by Codecov. Last update c3abb24...d2355a4

@shendepu shendepu force-pushed the fix-initial-state-with-async-reducer-when-create-store branch from 29074ac to d2355a4 Compare October 28, 2016 03:24
@fdlk
Copy link

fdlk commented Nov 4, 2016

Nice! We were also running into this issue. Took us a long time to debug.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants