-
Notifications
You must be signed in to change notification settings - Fork 71
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
相談部屋のコメントを省いたコメント数を出力するようにした #8223
base: main
Are you sure you want to change the base?
Conversation
@sugiwe |
@Ryooo-k |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
お疲れ様です!
コメント連投になってしまいましたが、ちょうど確認する時間が取れたので確認いたしました!
挿入コメントの方ご確認ください🙏
また、現時点でまだ Assignees のところにご自身の設定をされていなかったようなので、設定しておくと良いかと思います👍
@@ -18,6 +18,8 @@ class Comment < ApplicationRecord | |||
|
|||
mentionable_as :description, hook_name: :after_commit | |||
|
|||
scope :without_talk, -> { where.not(commentable_type: 'Talk') } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
新しくスコープを定義して「相談部屋以外のコメント」を抽出しやすくしたのが分かりやすくて良いなと思いました!
一方、今回の PR で表示数を揃えようとしているもう片側の「ユーザーページの上部タブにあるコメント(既に相談部屋のコメントは省かれている)」の数値取得の方( app/helpers/page_tabs/users_helper.rb
内)でも where.not(commentable_type: 'Talk')
の表記がこのまま使われているようでしたので、 users_helper.rb
内の方も without_talk
スコープに置き換えて良さそうに思ったのですが、いかがでしょうか?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
確かにおっしゃる通りです!ご指摘ありがとうございます!
スコープに置き換えます👍
@sugiwe |
@Ryooo-k |
@Ryooo-k where.notなどのスコープを組み合わせると以前はActiveRecordにバグがあったりして問題が起きがちなので、こちらの場合のテストが欲しいかなと思います。 |
@komagata |
Issue
概要
現状では、タブ内のコメント数とプロフィールページ内のコメント数が相違する場合があります。
変更後は、相談部屋のコメント数は省いて出力するようにしました。(komagataさんに確認済み)
変更確認方法
bug/fix-comment-count-mismatch
をローカルに取り込むi.
git fetch origin pull/8223/head:bug/fix-comment-count-mismatch
ii.
git checkout bug/fix-comment-count-mismatch
foreman start -f Procfile.dev
でローカルサーバーを立ち上げるkimura
、パスワードtesttest
でログインするScreenshot
変更前
変更後