Default handler about reduxsauce HOT 7 CLOSED

This is a really neat suggestion. I’m getting hung up a bit on the the DEFAULT name. Maybe PASSTHROUGH or NESTED or ELSE or maybe null?

Thoughts?

from reduxsauce.

I'm not sure if nested is the only one use case.. for example the default handler could do something like:

const defaultHandler = (state, { type, payload }) => {
  if (type.indexOf('FOO_') === 0) {
    return { ...state, fooActionsCount: state.fooActionsCount + 1 };
  }
  return state;
}

This is certainly bad code, but just as idea, that default handler is not always delegate to nested reducers. I think DEFAULT or ELSE could work. But in "standard" reducer style you usually use default keyword to mention, that this block handles all actions that are not mentioned above. So I think DEFAULT should be the best name for the constant.

from reduxsauce.

I see where you’re coming from.

I’ve never written anything like that but I guess that could be useful.

I’d be ok with this. You up for a PR?

from reduxsauce.

done :)

from reduxsauce.

Nice! I've been researching redauxsauce for this case and was going to make PR like yours :)
I hope @skellock will merge it and publish soon this feature.

P.S. Also I'd like to ask @skellock why reduxsauce has redux package in dependencies. I didn't see any usages of redux package in code. Am I something losing sight?

from reduxsauce.

@skellock is there forecast when npm build with this will be released?

from reduxsauce.

Sorry for the delay. I have it merged into master, but wanted to test a few things to make sure we're good. I don't have a good ETA other than once these other fires are extinguished. =)

from reduxsauce.