pyroscope icon indicating copy to clipboard operation
pyroscope copied to clipboard

Move error notifications to the caller

Open eh-am opened this issue 2 years ago • 0 comments

Currently we handle error notifications in the thunk, which turns out to not be the best design, since it's really the caller that knows best how to handle a failure (most likely a component)

Here's an example from the redux toolkit docs of how to do it. image (40)

This nos not super important right now, as a rule of thumb: if possible, refactor it, but don't go out of your way to do it

eh-am avatar Apr 06 '22 21:04 eh-am