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.

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

非表示になっている古いコメントの数を表示するようにしました #3517

Merged
merged 3 commits into from
Nov 11, 2021

Conversation

kotakawase
Copy link
Contributor

変更前

  • 古いコメントを表示する部分の文面が下のようになっているので、コメントの数を表示させて分かりやすくさせたい

image

変更後

  • コメントの数を表示させて、古いコメントが何件あるか分かるようにしました

image

補足

本PRの確認をする際に、rails db:seedを実行することで変更後で実装した表示を確認するためのテストデータが作成されます。これについては元々あったテストデータを活用させていただきました。
詳細は、db/fixtures/comments.ymlファイルの96行目に記述されています。

# db/fixtures/comments.yml
..
<% (1..20).each do |i| %>
comment<%= i + 16 %>:
  user: komagata
  commentable: product10 (Product)
  description: <%= "提出物のコメント#{i}です。" %>
  created_at: <%= Time.current + i.minutes %>
  updated_at: <%= Time.current + i.minutes %>
<% end %

上記のコマンドを実行後に以下の提出物にアクセスすると表示の確認をすることができると思います。
http://localhost:3000/products/1033498648

@kotakawase kotakawase self-assigned this Nov 4, 2021
@kotakawase kotakawase force-pushed the feature/show-the-number-of-old-comments branch from 6cf1a19 to c9bfe98 Compare November 5, 2021 05:48
@kotakawase
Copy link
Contributor Author

@ShotaTsuru さん
お手数ですがこちらレビューお願いいたします🙇‍♂️

@kotakawase kotakawase requested a review from ShotaTsuru November 5, 2021 06:06
@ShotaTsuru
Copy link
Contributor

@kawase-k 今現在レビューできる状態にないので申し訳ありあせんが、他の方にアサインいただいてもよろしいでしょうか?

@kotakawase
Copy link
Contributor Author

@ShotaTsuru さん
上記の件、承知しました!

もし不明な点をご共有いただけるのであれば僕としてもこういう意図があって実装したとお伝えすることができると思いますので、よろしければご連絡していただければと思います🙏

@kotakawase kotakawase removed the request for review from ShotaTsuru November 9, 2021 10:20
@kotakawase
Copy link
Contributor Author

@Ichiyo14 さん
お手数ですがこちらレビューお願いいたします🙇‍♂️

@kotakawase kotakawase requested a review from Ichiyo14 November 9, 2021 10:21
@Ichiyo14
Copy link
Contributor

Ichiyo14 commented Nov 9, 2021

@kawase-k
確認しました。実装はいいと思います!

変更されたテストの名称が'clicking "show old comments" will display old comments'となっているところに若干、違和感を感じました。変更前は古いコメントを表示するをクリックしていたのですが、変更後はそうなっていないので・・・。テスト名の修正をお願いします🙏

@kotakawase kotakawase force-pushed the feature/show-the-number-of-old-comments branch from c9bfe98 to 1deb810 Compare November 10, 2021 01:43
@kotakawase
Copy link
Contributor Author

kotakawase commented Nov 10, 2021

@Ichiyo14 さん
ご確認いただきありがとうございます🙏
テストの名称の件、修正してみましたので再度ご確認のほどお願いいたします!

@Ichiyo14
Copy link
Contributor

@kawase-k
確認しました! OKです✨

@kotakawase
Copy link
Contributor Author

@Ichiyo14 さん
レビューしていただきありがとうございました👌

@komagata さん
こちらのレビューをお願いいたします🙇‍♂️

@kotakawase kotakawase requested a review from komagata November 11, 2021 05:54
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 08926fc into main Nov 11, 2021
@komagata komagata deleted the feature/show-the-number-of-old-comments branch November 11, 2021 07:37
@github-actions github-actions bot mentioned this pull request Nov 11, 2021
22 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants