企業に所属しているアドバイザーのトップ画面に同じ企業の研修生リストを表示する

[yuma-matsui]

Issue

• https://github.com/fjordllc/bootcamp/issues/5215

Description

企業に所属しているアドバイザーのトップページに同じ企業の研修生のリストを表示。

確認方法

1. feature/add-training-subordinate-lists-at-adviser-pageブランチをローカルに取り込む
2. rails sで立ち上げる
3. senpaiでログインする
4. http://localhost:3000 にアクセスする

変更点

変更前

変更後

[yuma-matsui]

@keiz1213 お疲れ様です！水曜日からチーム開発に加わったyumです。 よろしくお願いいたします😌

こちらのレビューをお願いいたします🙏

[yuma-matsui]

@keiz1213 ご確認いただきありがとうございます！

ただ今回書いていただいたテストではカバー率が高くてテストが重くなってしうのかなと思いました。

確かにテスト回していてかなり遅いなと思いました😅 テストを書くか否かの判断ができなかったので大変参考になりました。

今回はご提案の通り下記2点のテストのみ残して他は削除しようと思います。

• 企業に所属するアドバイザーのダッシュボードには研修生が表示されている
• 一般の生徒のダッシュボードには表示されない

また、モデルのテストを書くことをすっかり忘れていたので上記修正と合わせて追加します。

[yuma-matsui]

@keiz1213 ご指摘いただいた箇所の修正とUserモデルのテストを追加したので再度ご確認をお願いします🙏

[yuma-matsui]

@keiz1213 ありがとうございます！！ 修正コミットを追加しましたのでご確認をお願いします🙏

[yuma-matsui]

@komagata お疲れ様です！ メンバーのレビューが済みましたのでご確認をお願いします。

[yuma-matsui]

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

一点確認ですが、修正コミットを作る際にmainブランチをリベースしてしまいコミット履歴を汚染してしまったのですが、 この場合はきれいにしてPRを作り直した方が良いでしょうか。

[komagata]

@yuma-matsui

はい、PRは必要なmainとの差分のみの状態でお願いします〜（PRを削除しなくても、修正で大丈夫です。）

[yuma-matsui]

@komagata 修正が完了しましたのでご確認をお願いします🙏

OKの場合はmachidaさんにデザインの依頼をさせていただきます！

[yuma-matsui]

@komagata ご指摘いただいた部分について修正してみました！！ scopeではなくメソッドでcollegue_traineesを取得する方向に転換してみました。

ご確認をお願いします🙏

[yuma-matsui]

@komagata こちらご指摘の箇所を修正しましたので再度レビューをお願いします。

[yuma-matsui]

@komagata ご指摘箇所を修正しましたのでご確認をお願いいたします。

[yuma-matsui]

ご確認いただきありがとうございます！！

@machida こちらのデザインをお願い出来ますでしょうか。