ユーザー一覧に「コース別」のページを追加した

ham-cap commented 2 months ago

Issue

7998

概要

変更確認方法

feature/add-a-new-tab-in-which-users-are-sourted-by-courses-to-the-users-indexをローカルに取り込む
http://localhost:3000/users にアクセスし、「コース別」タブが追加されていることを確認
「コース別」タブをクリックし、「Railsエンジニアコース」所属ユーザーの一覧が表示されることを確認
絞り込み検索ができるか確認（例えば、「marumaru」と入力するとマルマル企業所属のユーザーのみに絞り込めます）
「フロントエンド」のタブをクリックし、「フロントエンドエンジニアコース」所属ユーザーのみの一覧が表示されることを確認
「その他」のタブをクリックし、「Unityゲームエンジニアコース」及び「iOSエンジニアコース」所属ユーザーのみの一覧が表示されていることを確認する

Screenshot

変更前

変更後

ham-cap commented 2 months ago

@goruchanchan お疲れ様です！こちらのPRにつきまして、レビューをお願いできればと思います🙏 急ぎではありませんのでお手隙の際で大丈夫ですが、もしご都合が悪いようであれば遠慮なくお知らせください👍

goruchanchan commented 2 months ago

@ham-cap レビュー依頼ありがとうございます！ぜひやらせていただければと思います！一週間程度で確認いたしますので今しばらくお待ちください🙇‍♂️

ham-cap commented 2 months ago

@goruchanchan ご指摘いただいた点について修正いたしましたので再度ご確認いただければ幸いです🙏

ham-cap commented 1 month ago

@goruchanchan こちら修正いたしましたので再度ご確認いただければ幸いです🙏

ham-cap commented 1 month ago

@goruchanchan ありがとうございます🙏 お付き合いいただきありがとうございました😃

@komagata こちらレビューをお願いできますでしょうか🙏 一点、ユーザーをコース別に絞り込むために使用するCOURSE_NAMESというコース名のハッシュをCourseモデルの中に定義し、Users::CoursesControllerから参照しているのですが、Courseモデル自身が具体的なコース名であるCOURSE_NAMESを持っているのが不自然ではないかどうかご教示いただけないでしょうか🙇‍♂️

最初はUsers::CoursesController内でローカル変数として定義して使用していたのですが、コードレビュー中にgoruchanさんからモデルに書いた方が自然ではないかとの指摘を受け、現在はCourseモデル内に書いております。考えようによってどちらでもおかしくないように思えてきてしまい、結局どこに書くのが一番自然なのかがわからなかったため、考え方をお示しいただければ幸いです🙇‍♂️

ham-cap commented 1 month ago

@komagata ご指摘いただいた点について修正してみましたのでご確認をお願いいたします🙏

fjordllc / bootcamp