在Redux中处理异步操作错误的最佳实践是什么？

Question

export const saveSomething = (thing = {}) => {
  dispatch({
    type: THING_SAVING,
  });

  return async function (dispatch) {
    try {
      await persistThing(thing);
    } catch (e) {
      dispatch({
        type: THING_SAVE_ERROR,
        error: e,
      });
      throw e;
    }

    dispatch({
      type: THING_SAVED,
      error: e,
    });
  }
}

handleSubmitClick(e) {
  dispatch(saveSomething({ name: e.target.value }))
    .catch(e => {
      // pass
    });
}

So, I'm probably overthinking this, but I'm trying to figure out the best practice here. When my async action has an error, even though I'm dispatching an error, it still feels right to throw it so the promise is rejected in case some components want to key on that. But, then that means in order to not have Create-React-App bomb on an unhandled promise rejection, I need to add this dummy catch when I dispatch the action, which IMHO is kinda ugly.

What's the best practice here? Not throw the error? Use the dummy catch? Something else?

Edit: The reason I need the dummy catch is because otherwise the app is bombing on me, at least in create-react-app:

(the error doesn't 100% match my pseudo code, but you get the point)

Answer 1

In my own projects I would try to manage all the error applications, probably with some generic action dispatcher that could be modified with proper error codes. But as for your example, you might not want to throw the error over, because you are using await in the method that can cause the error. So I would rewrite the code like this:

export const saveSomething = (thing = {}) => {
  dispatch({
    type: THING_SAVING,
  });

  return async function (dispatch) {
    try {
      await persistThing(thing);
      dispatch({
       type: THING_SAVED,
      });
    } catch (e) {
      dispatch({
        type: THING_SAVE_ERROR,
        error: e,
      });
    }
  }
}

If nothing happens and everything goes through the happy path, after the completion of the async call persistThing , the action dispatcher would get executed. And if persistThing generates an exception, the error action dispatcher would get executed.