2回連続で今日の気分がsadの日報を表示する機能をVueからサーバーサイドレンダリングに変更

[hagiya0121]

Issue

• sad_reports.vueを非Vue/React化したい #8227

概要

2回連続で日報の今日の気分がsadの生徒をメンターのダッシュボードに表示する機能があります。
Vueで実装されていたその機能をサーバーサイドレンダリングに変更しました。

変更確認方法

1. feature/sad-reports-non-vueをローカルに取り込む
   1. git fetch origin pull/8237/head:feature/sad-reports-non-vue
   2. git checkout feature/sad-reports-non-vue
2. foreman start -f Procfile.dev でローカルサーバーを立ち上げ
3. 好きなユーザーでログイン
4. 今日の気分がsadの日報を作成する
   日報の新規作成
5. もう一度、今日の気分がsadの日報を作成する
6. メンターのアカウントでログインしてダッシュボードにアクセス
7. スクリーンショットの画像のように2回連続sadのユーザーという項目が表示されていることを確認

Screenshot

[hagiya0121]

@kyokucho1989
お疲れ様です。
こちらのPRのレビューをお願いしたいです🙏
ご都合が悪いときは遠慮なくおっしゃってください🙇

[kyokucho1989]

@hagiya0121

了解しましたー
一週間ほどお待ちください

[kyokucho1989]

app/views/api/reports/sad_streak/index.json.jbuilder のファイルも不要になったと思うので削除してください。

#5621
これがVue化したPRなので、これを参考にファイルを探しました。

[kyokucho1989]

@hagiya0121
レビュー終了しました。動作は問題ありません。不要なファイルがまだ一点あったので、それを削除していただければいいかと思います。

[hagiya0121]

@kyokucho1989
レビューありがとうございます🙏修正したので確認お願いします。
vueで実装したときのPRをたどる方法は思いつかなかったので助かりました！

[kyokucho1989]

問題ありません。approveしますー

[hagiya0121]

@kyokucho1989
レビューありがとうございました🙏

[hagiya0121]

@komagata
メンバーのレビューが完了したので確認お願いします🙏

[komagata]

@okuramasafumi こちらレビューお願い致します〜

[okuramasafumi]

おお、こんなにコード消せるんですね、すごい！
削除されたコントローラー内での@reports変数の中身が

bootcamp/app/models/user.rb

Lines 484 to 492 in 9db2507

	def depressed_reports
	ids = User.where(
	hibernated_at: nil,
	retired_on: nil,
	graduated_on: nil,
	sad_streak: true
	).pluck(:last_sad_report_id)
	Report.joins(:user).where(id: ids).order(reported_on: :desc)
	end

と同じであることを確認しました。

ところで、このdepressed_reportsメソッド、内容的にはReportモデルにあってほしい気がしますね。今後のリファクタリングのネタにできそうです。