redux-optimist dependency causes wasteful mapStateToProps invocations

[aduth]

This past summer I had reported an issue at ForbesLindesay/redux-optimist#22 . In reviewing the behavior of react-redux recently, I found that it is smart enough to avoid calling each connected components' mapStateToProps if the reference for state does not change after a dispatched action. This should ideally occur if a dispatched action causes no change in state value. However, with redux-optimist, whether or not a change occurs, a new object reference will be returned upon a dispatch, therefore causing mapStateToProps for every single connected component to be invoked.

We should either move away from redux-optimist, or submit an upstream patch, as given the lack of response on ForbesLindesay/redux-optimist#22 since June, it is unlikely to be otherwise resolved.

[aduth]

Pull request with proposed fix submitted at ForbesLindesay/redux-optimist#25