Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

プロフィール編集画面 #10

Merged
merged 8 commits into from
Sep 21, 2023
Merged

プロフィール編集画面 #10

merged 8 commits into from
Sep 21, 2023

Conversation

KOH6
Copy link
Owner

@KOH6 KOH6 commented Sep 20, 2023

課題のリンク

やったこと

  • プロフィール編集画面の作成を行いました。

動作確認方法

その他

  • rails_best_practicesの指摘でapp/controllers/users_controller.rb#L13にて、use scope accessの指摘が発生しておりました。解消方法が分からず解消できておりません。
  • 今回は「URLパラメータのidがログインユーザ以外の場合は編集画面に遷移しない」という内容が実装したかったものになります。以下指摘が該当するようなのですが、あるレコードから参照を辿っている際に発生する指摘のようで修正方法が分からずでした。

@KOH6 KOH6 self-assigned this Sep 20, 2023
Copy link
Collaborator

@yamaryo416 yamaryo416 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

こちらに関して、Userモデルにインスタンスとして切り出すとどうですかね?
難しい場合は、無視してもいいかなと思いました。
(自分のユーザー情報のみ更新するので、:idを受け取らないroutingが望ましいが、別のcontrollerを作るとなるとRESTから逸脱するので、悩ましいところ)

@KOH6
Copy link
Owner Author

KOH6 commented Sep 21, 2023

仕様にも明言されていない箇所ですし、rails_best_practicesのuse scope accessの指摘は解消難しいので無視する形でいきます!
コメントありがとうございます!マージいたしますね!

@KOH6 KOH6 merged commit 7972774 into main Sep 21, 2023
10 checks passed
@KOH6 KOH6 deleted the profile_edit branch September 21, 2023 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants