refactor to use form_tag instead of form_for