Let Users Change Their Email: Code Review Practice

Support gets a few requests a week from people who want to change the email on their account. This adds an update action to AccountsController, wired to the existing form on the account page, plus an email format and uniqueness validation on User. There's a request spec for the happy path.

Mapp/controllers/accounts_controller.rb1 of 3

Files Changed

app/controllers/

app/models/

spec/requests/

app/controllers/accounts_controller.rb

Ruby · Click a line number to comment · Tap a line to comment

@@ -1,7 +1,16 @@

class AccountsController < ApplicationController

before_action :require_login

def show

@user = current_user

end

def update

@user = User.find(params[:id])

if @user.update(params.require(:user).permit!)

redirect_to account_path, notice: "Your email was updated."

else

render :show, status: :unprocessable_entity

end

app/controllers/accounts_controller.rb

app/models/user.rb

spec/requests/accounts_spec.rb