My preference is that if a function ever does return something;, then every code branch should have an explicit return something;

(Does not apply if the only return in a function is a simple return;)

I don't have a strong preference and having an implicit return undefined is fine for me - but we have a convention so it's best to keep it.

Regarding @atul suggestion, I don't agree it is a good idea. Adding an if at the beginning will introduce a duplication in our logic. If we ever decide to have a new branch, we would need to update two places - so keeping it at sync would be more expensive.

Early returns are generally advantageous, but only if they simplify the logic and mental effort. They shouldn't be the goal on their own.

One possible option would be to create an object with status as a key and function that returns a React.Node as a value. Then we can have an early return without compromising the maintainability, but I don't think it is worth it.