refactor signin form to use form_tag instead of form_for