We only show a success message when there is a clear signal to do so.
Avoid “assumptive” success messages unless very clear reason.
A request does not need to show a success message by default.
If it does show one it must have a high degree of correctness.
Showing success on 200 OK is good.
Showing success because another process completed is bad because it assumes success when in fact that process may have failed.
A request must propagate errors to the UI unless there is a clear reason not to.
We will reject any pull requests that don’t satisfy this.
Good: Catching error and passing to regular error component.
Bad: Doing nothing with error or throwing it to console log.
Updated 3 months ago