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

Alternative combination of top-level reducer (docs) #194

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

Alternative combination of top-level reducer (docs) #194

wants to merge 1 commit into from

Conversation

dpwrussell
Copy link

It seems likely that most people will have an app reducer already combined at the point at which the store is created, having imported their top-level precombined reducer.

Happy to reformulate this into an alternate section as opposed to a comment or whatever you suggest. I think the current example will likely lead people to introducing an extra level into their reducers that they did not want.

I can't think of a cleaner way than this right now, but you get the idea.

It seems likely that most people will have an app reducer already combined at the point at which the store is created, having imported their top-levelprecombined reducer.
Happy to reformulate this into an alternate section as opposed to a comment or whatever you suggest. I think the current example will likely lead people to introducing an extra level into their reducers that they did not want.
@codecov-io
Copy link

codecov-io commented Jun 23, 2017

Codecov Report

Merging #194 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #194   +/-   ##
=======================================
  Coverage   99.06%   99.06%           
=======================================
  Files          19       19           
  Lines         215      215           
=======================================
  Hits          213      213           
  Misses          2        2

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4dacfe8...39177d4. Read the comment docs.

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.

2 participants