Feature request use mapStateToProps in mapDispatchToProps

Hello

First of all I would thank you for this awesome library been using it in all my projects lately!

There is just one thing that is bugging me. I often would like to use the state props I pick from the mapStateToProps in mapDispatchToProps. Right now I use a library called recompose and a hoc called withHandlers to do this.

Example how I do today.

compose(
  connect(
    (state) => ({
      someProp: state.someState,
    }),
    (dispatch) => ({
      onSomeAction: (someProp) => dispatch(someAction(someProp)),
    }),
  ),
  withHandlers({
    onSubmit: ({ someProp, onSomeAction }) => (e) => {
      e.preventDefault();
      onSomeAction(someProp);
    },
  }),
)(DumbComponent);

Suggested feature.

connect(
  (state) => ({
    someProp: state.someProp,
  }),
  (dispatch) => ({
    // If the dispatch prop is executed and returns a functions then run that function automatically with the latest props
    onSubmit: ({ someProp }) => (e) => {
      e.preventDefault();
      dispatch(someAction(someProp));
    },
  }),
)(DumbComponent);

[timdorr]

I’d just use the getState arg in redux-thunk, personally. Way more testable that way.

[markerikson]

Mmm… I may be mixing up cases in my head between using the object shorthand and using an actual mapDispatch function. I’ll have to look at this again later.

I still stand by my point that I don’t think this is worth another edge case.