Comment by simonw

I don't find this easier to read:

    !(await userService.getUserByEmail(user.email)) || throwError(err.userExists);

I guess if I worked in a codebase that used that pattern consistently I'd get used to it pretty quickly, but if I dropped into a new codebase that I didn't work on often I'd take a little bit longer to figure out what was going on.