Consider Changing Public API about redux-promise-middleware

[pburtchaell]

Update

Thanks to a PR by @rahulbdominic and initial from @zhanyuzhang, we have this on master. Will work on a major version release soon.

[zhanyuzhang]

Maybe you should check the arguments first. Just as follows:

export default function promiseMiddleware(config = {}) {
  let { dispatch, promiseTypeSuffixes, promiseTypeDelimiter } = config;
  const PROMISE_TYPE_SUFFIXES = promiseTypeSuffixes || defaultTypes;
  const PROMISE_TYPE_DELIMITER = promiseTypeDelimiter || '_';

  const fn = next => action => {
    // the main code comming here....
  }

  // if 'dispatch' in the 'config' object, return the main code directly.
  if(dispatch) {
    return fn;
  }

  // else return the middleware function
  return ref => {
    dispatch = ref.dispatch;
    return fn;
  }
}

[pburtchaell]

Update

There is an open PR by @zhanyuzhang at #223.

[rahulbdominic]

@pburtchaell I noticed this has been inactive for a while. Is it alright if I pick this issue up?

[pburtchaell]

Yes! Feel free to make a PR. Maybe use #223 as a starting point—had some feedback on that PR and it was never addressed, so I closed it.

[pburtchaell]

Closed with release of version 6.0.0.