Merge pull request #276 from jmmastey/fix-password-reset-path

Awesome @jmmastey. I think we went with a match route, later changed it as match was *sorta-ish* deprecated in Rails 4+. Anyways, believe those changes might have caused some issues.

Either way, verified everything worked locally and performed PR. Thanks again!
This commit is contained in:
Ken Johnson
2017-10-11 11:20:15 -04:00
committed by GitHub
2 changed files with 5 additions and 2 deletions
@@ -25,7 +25,7 @@ class PasswordResetsController < ApplicationController
end
end
def forgot_password
def send_forgot_password
@user = User.find_by_email(params[:email]) unless params[:email].nil?
if @user && password_reset_mailer(@user)
+4 -1
View File
@@ -3,9 +3,12 @@ Railsgoat::Application.routes.draw do
get "login" => "sessions#new"
get "signup" => "users#new"
get "logout" => "sessions#destroy"
post "forgot_password" => "password_resets#forgot_password"
get "forgot_password" => "password_resets#forgot_password"
post "forgot_password" => "password_resets#send_forgot_password"
get "password_resets" => "password_resets#confirm_token"
post "password_resets" => "password_resets#reset_password"
get "dashboard/doc" => "dashboard#doc"
resources :sessions do