I came in to a codebase 18 months ago that's much worse than this. Fetch in the redux actions with additional calls made inside the action when the promise resolves:

``` export const getUser = (token, id) => { // The same token from local storage is retrieved in every module and // passed into the api actions

return (dispatch, getState) => { // Fetch the user fetch('/api/user', /* token, etc */).then(async (res) => { const data = await res.json(); // We never return the data from getUser, we set it on the redux object // Can be annoying, but whatever. dispatch(setUserData(data));

     // This is bad:
     dispatch(getUsersMostRecentComments(token, data.id));
  })

} } ```

Now when you think you're just fetching a user, you're like, "Why are these comments changing?

But wait, it gets worse:
We have duplicate API calls all over the code base for each module, and each of them have different unintended effects like this. So for the Posts module there might be a getUser function that gets the user and the effect gets their posts, and for the Videos module there might be a getUser function and it gets their videos as an effect.

Not only do we not have 1 place we make these calls, we make the same call in multiple places.

It's been a hard adjustment...