アドバイザーのダッシュボードに現役の研修生の最新10件の日報を表示する機能を追加

[pachikuriii]

Issue

• #5189

概要

アドバイザーでログインし、なおかつそのアドバイザーと同じ企業に所属する現役の研修生達がいた場合、その研修生達が書いた日報の最新10件を研修生の最新の日報としてダッシュボードで表示されるようにしました。

変更点

変更前

変更後

新たに研修生の最新の日報が表示されます。

変更確認方法

1. ブランチfeature/display-trainees-the-latest-daily-reports-on-advisors-dashboardをローカルに取り込む
2. 本PRでdb/fixtures/reports.ymlへ追記を加えたため、DBへ反映させるためにbin/rails db:resetを実行する
3. bin/rails sでローカル環境を立ち上げる
4. advijirouでログインし、登録情報変更ページ( http://localhost:3000/current_user/edit ) へアクセスする
5. 一番下までスクロールし、所属企業をMaruMaru Inc.に設定しadvijirouのプロフィール情報を更新する
6. ダッシュボード( http://localhost:3000/ )にアクセスし、MaruMaru Inc.に所属する現役の研修生の最新10件の日報が表示されることを確認する

[pachikuriii]

@machida

こんにちは🍵こちら、デザインをお願いできますか…🙏？

• もっと見るのリンクは現状では最新のお知らせと同様
• 本PRで作成した現役の研修生の日報が表示される位置は最新のお知らせと研修生の間

現状のコードでは上記のようにしています。 ご確認をよろしくお願いします🌈

[machida]

@pachikuriii デザイン了解ですー💪

[machida]

@pachikuriii おまたせしました！デザイン入れました。

@komagata レビューお願いします🙏

[machida]

@komagata すいません、受講生レビューがこれからでした🙏

[machida]

@akingo55 レビューお願いします🙏

[akingo55]

@machida 表示は問題ありませんでした！ 1点コントローラーの実装でコメントしました！

[pachikuriii]

対応が遅くなり、すみません😭🙏

@machida 町田さん、デザインとレビューの依頼をありがとうございます🌷

@akingo55 akinaさん、レビューをありがとうございます！ インスタンス変数の定義を2つに分けていたのは、1行にしてしまうと変数に代入するコードが長すぎる🔥👊🏻とrubocopに指摘されてしまうのことが背景にあります。 akinaさんにご提案いただいたように62行目の方はインスタンス変数にせず、1行で変数を定義する形へ変更しております。 お手すきの際に再度レビューいただけますとうれしいです☺️

[machida]

@pachikuriii

最新の main から rebase したので、このブランチをいじる前に

git pull --rebase origin feature/display-trainees-the-latest-daily-reports-on-advisors-dashboard

をお願いしますー

[pachikuriii]

@akingo55 丁寧にレビューいただき、ありがとうございます！🌷

@machida かしこまりました✨ご連絡ありがとうございます🙏

@komagata akinaさんよりApproveいただきましたので、お手隙の際にレビューをよろしくお願いします☺️

[machida]

コンフリクトを解消したのでマージしますー

[pachikuriii]

@machida コンフリクト解消いただき、ありがとうございます🙏！