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.

Sign up for GitHub

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

Jump to bottom

提出物に提出者最終コメント日時とメンター最終コメント日時を追加する #3111

Merged
merged 18 commits into from
Sep 8, 2021
Merged

提出物に提出者最終コメント日時とメンター最終コメント日時を追加する #3111

merged 18 commits into from
Sep 8, 2021

Conversation

Daichi831
Copy link
Contributor

@Daichi831 Daichi831 commented Aug 10, 2021 •
edited
Loading

issue #3002

概要

提出物にself_last_comment_at,mentor_last_comment_atの2つのカラムを追加する

それぞれは提出物に対する提出者の最終コメント日時、メンターの最終コメント日時です。
表示されるのは両者のうち最新の日時が表示されます。
この機能により提出物を確認した際にメンターのコメント待ちなのか判断が簡単にできるようになります。

  • 一覧ページ
    貼り付けた画像_2021_08_10_20_53

  • 個別ページ
    貼り付けた画像_2021_08_10_20_47

既存の提出物にデータを挿入するdata migrationスクリプトを作成

既存の提出物にはコメントがついていますがカラムに保存されるのはこの機能の実装後にされたコメントに対してです。
そこで既存の提出物のカラムにもデータを挿入するためのdata migrationスクリプトを作成しました。

@Daichi831 Daichi831 force-pushed the feature/add_last_comment_date_and_time_column_to_product_model branch from cd2a1ad to 288b7ee Compare August 10, 2021 08:07
@machida
Copy link
Member

machida commented Aug 11, 2021

📝
メンターか本人どちらか最終のコメントだけを出すように変更になった

@Daichi831 Daichi831 self-assigned this Aug 11, 2021
@Daichi831 Daichi831 force-pushed the feature/add_last_comment_date_and_time_column_to_product_model branch from 0029df7 to 2512731 Compare August 17, 2021 11:39
Daichi831 added 15 commits August 19, 2021 00:19
@Daichi831
提出物に提出者最終コメント日時とメンター最終コメント日時のカラムを追加
c63d466
@Daichi831
コメント後に最終コメント日時を更新するメソッドをCommentCallbacksに追加
c18628a
@Daichi831
提出物個別ページに最終コメント日時があれば表示するように実装
af6224d
@Daichi831
提出物一覧ページに最終コメント日時があれば表示するように実装
3d468a4
@Daichi831
rubocopによる修正
c793342
@Daichi831
products/showにあった空白を削除
289f7fc
@Daichi831
prettierでproduct.vueを修正
b717cbb
@Daichi831
提出物個別ページと一覧ページの見た目を修正
27adf33
@Daichi831
提出者とメンターのコメントの最新の方を表示するように変更&デザインようにクラスを追加
1d45737
@Daichi831
prettierでproduct.vueを修正
6889b3b
@Daichi831
不要な空白の削除
028f745 
空白があったため削除
@Daichi831
コメントが削除された時最新日時を更新する
e17ca84
@Daichi831
既存の提出物にデータを挿入するdata_migrationスクリプトを追加
eb70568
@Daichi831
rubocopによる修正
da0fc9c
@Daichi831
提出物にコメントがされた際に最終コメントが更新されるか検証するテストを追加
f7a9190
@Daichi831 Daichi831 force-pushed the feature/add_last_comment_date_and_time_column_to_product_model branch from 2c3cc6f to f7a9190 Compare August 18, 2021 15:20
@Daichi831 Daichi831 marked this pull request as ready for review August 18, 2021 17:00
@Daichi831 Daichi831 requested a review from yamadaab August 19, 2021 08:55
@Daichi831
Copy link
Contributor Author

@yamadaab
こちらお時間あるときにレビューをよろしくお願いします！🙇‍♂️

@yamadaab
Copy link
Contributor

@yamadaab
こちらお時間あるときにレビューをよろしくお願いします！🙇‍♂️

確認が遅れて申し訳ありません。本日中にレビュー致します🙇‍♂️

@yamadaab
Copy link
Contributor

yamadaab commented Aug 23, 2021 •
edited
Loading

@Daichi831

遅くなりすみません。大丈夫だと思います！🙇‍♂️

@Daichi831
Copy link
Contributor Author

@yamadaab
お忙しい中、ありがとうございます😭

@komagata
こちら確認をよろしくお願いします🙇

@Daichi831 Daichi831 requested a review from komagata August 23, 2021 04:35
@komagata
Copy link
Member

@Daichi831 すみません、僕としては、二つのカラムを追加するのは、一覧の表示を早くするのに必要なのでお願いしたいと思っていました。

ですので、見た目上の表示を増やすというのは考えていませんでした。

こちらは @machida さんと話してそうなった感じですかね？

@Daichi831
Copy link
Contributor Author

@komagata
先々週のミーティングの際にカラムを2つとも表示させていたのですが、情報量が多くなってしまうとのことでメンターか提出者のどちらか最新を出すようになりました。
駒形さんとしては表示までは考えていなかったということでしょうか？

@Daichi831
Copy link
Contributor Author

最終コメント日時の表示はそのままで進めることになりました。

@Daichi831
Copy link
Contributor Author

@machida
こちらデザインをよろしくお願いします！🙇‍♂️

@machida
Copy link
Member

machida commented Sep 6, 2021

@Daichi831 @komagata デザイン入れました。提出物個別ページには最終コメントを出さないようにしましたー。
テストが通ったらマージしますー

@machida machida force-pushed the feature/add_last_comment_date_and_time_column_to_product_model branch from 200add6 to dc522c8 Compare September 6, 2021 08:07
@machida
Copy link
Member

machida commented Sep 6, 2021

テスト通ったのでマージしますー

@machida
Copy link
Member

machida commented Sep 6, 2021

あ、まだレビューが通ってなかった。

@komagata レビューお願いします。
表示はほとんど同じで、最後のコメント日時のデータを今回作ったものを使うようにしました。

@machida machida removed their assignment Sep 6, 2021
komagata
komagata requested changes Sep 6, 2021
View reviewed changes
app/models/comment_callbacks.rb Outdated
# 最終日時をリセットする
product.mentor_last_comment_at = nil
product.self_last_comment_at = nil
# 最新日時を上書きする
Copy link
Member

Choose a reason for hiding this comment

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

コメントを入れるのであれば同じ意味のメソッドとしてしまうのがわかりやすいかもです〜

Copy link
Contributor Author

Choose a reason for hiding this comment

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

こちら最終日時のリセット部分はメソッドにしました〜

@Daichi831
Copy link
Contributor Author

Daichi831 commented Sep 6, 2021 •
edited
Loading

@machida
ありがとうございます😊
@komagata
こちら確認をよろしくお願いします🙇

komagata
komagata approved these changes Sep 8, 2021
View reviewed changes
Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

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

確認しました、OKですー🙆‍♂️

@komagata komagata merged commit b49de33 into main Sep 8, 2021
@komagata komagata deleted the feature/add_last_comment_date_and_time_column_to_product_model branch September 8, 2021 11:25
@github-actions github-actions bot mentioned this pull request Sep 8, 2021
Merged
14 tasks
@gentamura gentamura mentioned this pull request Sep 12, 2021
Closed
@IkumaTadokoro IkumaTadokoro mentioned this pull request Nov 10, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yamadaab yamadaab yamadaab approved these changes

@komagata komagata komagata approved these changes

Assignees

@Daichi831 Daichi831

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Daichi831 @machida @yamadaab @komagata