You don't need to change this, but I personally don't like that you named this helper ```login?```. I would've preferred ```is_logged_in?``` since it's more clear / descriptive.