I like not letting users who aren't signed in see this page, but what would you think about putting that into a ```before_action``` instead? Perhaps name it ```check_for_logged_in_user``` or somesuch? That would make this method do one thing and one thing only, and let you reuse the "redirect if not logged in" logic.