Remove usage of bindActionCreators

[ZeshanA]

Currently, the codebase uses bindActionCreators in conjunction with mapDispatchToProps. This may be unnecessary, see Option 4 here and this article

[ZeshanA]

So this is definitely not best practice, as mentioned in the React-Redux documentation here.

We should swap all mapDispatchToProps to be defined using object shorthand, we don't need the additional control over dispatch offered by defining it as a function. Will do this manually at some point, it isn't critical.

mapDispatchToProps as a function

const mapDispatchToProps = dispatch => bindActionCreators({
    createNewRegion: createNewRegion,
    handleFormSubmit: handleFormSubmit
});

mapDispatchToProps as an object (recommended)

const mapDispatchToProps = {
    createNewRegion,
    handleFormSubmit
}