8.5.1 refactor signin form to use form_tag