Facebook認証で登録したUserが後からパスワードを設定できるようにする

[kudojp]

このPRはfacebook認証によるサインインとログイン機能の追加 #4の続きである。

仕様

• 現在のprofile更新ページを2つに分ける。

  • :name, :is_male, :height, :weight, :commentを更新するページ。ここではパスワードの入力を求めない。
  • passwordを更新するページ。ここではcurrent password, new password, new pass confirmationの3つの入力が求められる。ただし、facebook認証で登録して、まだpasswordを登録していないuserは、current passwordの入力を求められない。

• ヘッダーのドロップダウンメニューを以下にする

  • My Profile
  • Update Profile(→押すとprofile更新ページへ)
  • Update Password(→押すとpassword更新ページへ)
  • Log Out (→押すと「ログアウトしますか？」とalart表示される)
  • Sign Out (→押すとSignOut確認ページへと飛ばされる)

• facebook認証したuserが後からpasswordを設定することができるようにする

設計

今回の機能で使われるエンドポイント

• profile更新ページ
  /profile/edit GET users#edit_profile
• profile更新のPUT,POST先のendpoint
  /profile PUT,POST users#update_profile
• password更新のためのページ(passwordの設定有無でpageを出し分ける)
  /password/edit GET users#edit_password
• password更新のためのPUT,POST先のendpoint
  /password PUT,POST users#update_password
• userのログアウトのためのエンドポイント
  /sign_out DELETE devise/sessions#destroy
• 退会の確認ページ
  /cancellation/confirmation GET users#cancel_confirmation
• 退会時のdeleteリクエストの送信先
  /cancelletion DELETE users#destroy

除去されるエンドポイント

• 元来userのプロフィール変更と退会の機能を提供していたページ
  /users/edit GET devise/registrations#edit (未削除)
• 元来userのプロフィールとパスワード変更のPUT,POST先のエンドポイント
  /users PUT,POST devise/registrations#update　(未削除)

このPRのdev環境

TODO
devでDBのマイグレーションがうまくいかないのでstage環境にpushした。
https://diet-app-2020.herokuapp.com

機能の仕様

今後のPRで対応すること

• password認証したuserが後からauth認証を加えることができるようにする

質問

[kudojp]

今回のPRで(上記の通り)2つのエンドポイントを削除する必要がある。特に後者を消さないとemailとaccount_idが変更できてしまう。

• 元来userのプロフィール変更と退会の機能を提供していたページ
  /users/edit GET devise/registrations#edit
• 元来userのプロフィールとパスワード変更のPUT,POST先のエンドポイント
  /users PUT,POST devise/registrations#update

これを実現するため、これに習って、config/routes.rbにおいて、

  devise_for :users, skip: [:registrations], controllers: { omniauth_callbacks: 'users/omniauth_callbacks' }

  devise_scope :user do
    get 'users/cancel', to: 'devise/registration#cancel', as: :cancel_user_registration
    get '/users/sign_up', to: 'devise/registration#new', as: :new_user_registration
    delete '/users', to: 'devise/registrations#destroy', as: :user_registration
    post '/users', to: 'devise/registration#create'
  end

実質これによってこれらのエンドポイントが取り除かれたことになる。

               edit_user_registration GET      /users/edit(.:format)                                                                    devise/registrations#edit
                    user_registration PATCH    /users(.:format)                                                                         devise/registrations#update
                                      PUT      /users(.:format)                                                                         devise/registrations#update

[azumag]

よさげです。問題ありそうなところは見当たらなかった