Does the TODO make sense? Move what to the signal? If you put your code there it looks like the comment is talking about your `auth.login` call.