hirano-vm4

> @hirano-vm4 conflictの修正をお願い致します～。 @komagata こちらconflictを解消しました！よろしくお願いします🙏

@unikounio もちろん大丈夫です🙆お時間数日いただければと思います！よろしくお願いします🙏

@unikounio お疲れさまです！遅くなりました！早速コードを確認しました😄実装のコードは自体はわかりやすく、良いコードだと感じましたが、1点だけ気になった点を共有したいと思います🙏 ## ユーザー一覧での表示ズレ `http://localhost:3000/users/` - 問題ない表示 - 要素の高さがずれている表示 私の環境に固有の現象かもしれませんが、ブラウザのサイズを変更するとactivity-nameに設定された「ポートフォリオ」という文字が他の文字に比べて多いため、横幅がギリギリになっています。 その結果、ブラウザのウィンドウサイズによっては文字列が2行表示になり、レイアウトが崩れてしまうようです🧐 `app/javascript/components/user-activity-counts.vue` ```vue user-activity-count( activity-name='ポートフォリオ', :activity-count='user.work_count', :activity-url='`${user.url}/portfolio`') ``` 固有の問題の可能性もあるので @unikounio さんの環境でも念の為確認していただき、状況によっては町田さんにこのままいくかどうかの確認を入れても良いかも？と感じましたがいかがでしょうか？🙆 ## 確認チェック - [x] カウント数一覧に「ポートフォリオ」項目が追加されている - [x] カウントの数字部分のリンクが次の状態になっている -...

@unikounio お疲れ様です！もとからそのような状況だったのですね👀コード的には私は問題ないと思いましたので、確認できたらApproveさせてもらいます🙏

@dowdiness ## レビュー依頼 お疲れ様です🙏 お忙しい中、申し訳ありませんがレビューをお願いすることは可能でしょうか🙆？ 急ぎではありませんので、無理ないタイミングで全く問題ありません！ ご検討のほど、よろしくお願いいたします🙇‍♂️

@dowdiness お疲れ様です🙏 念の為の確認になります〜！ お忙しい中、申し訳ありませんがレビューをお願いすることは可能でしょうか🙆？ 急ぎではありませんので、無理ないタイミングで全く問題ありません！ ご検討のほど、よろしくお願いいたします🙇‍♂️ ## 追記 お忙しそうなので今回は別の方に依頼したいと思います！またお願いいたします〜

@kyokucho1989 お疲れ様です🙏 お忙しい中、申し訳ありませんがレビューをお願いすることは可能でしょうか🙆？ 急ぎではありませんので、無理ないタイミングで全く問題ありません！(1~2週間後とかでもOKです) ご検討のほど、よろしくお願いいたします🙇‍♂️

@kyokucho1989 ありがとうございます！おっしゃる通り、このIssueで作業の過程で発生した確認の必要のない修正まで細かくコミットしてしまっていたのでコミットをまとめました🙏 確認しにくい状態になっていて、お手数おかけしました！引き続きよろしくお願いします🙇‍♂️

@kyokucho1989 ありがとうございました🙏分かりにくい部分を確認いただき、ありがとうございました！引き続きよろしくお願いします！ @komagata ## レビュー依頼 チームメンバーによるレビューが終わりました！レビューをお願いします🙏 ## コードで確認したい点 現状、バグの修正は終了していますが、 `app/views/api/reports/index.json.jbuilder`から`_user_icons.json.jbuilder`に`report`を渡すとActiveDecoratorが適用され、`report.comments`を渡すと`primary_role`を取得できない部分がありました。 これはActiveDecoratorによるもので、関連モデルを別ファイルに渡すと自動デコレートされないことが原因です。 今回はデコレートされた`report`を渡すことで対応しましたが、それにともなって[このコミット](https://github.com/fjordllc/bootcamp/pull/7559/commits/ea4cd4cb07a8786d5082c5a86b0838593d2e423a)にあるように複数の`.jbuilder`も同様に修正しました。 しかし、明示的に以下のようにdecorateすると以下のファイルの一箇所だけの修正だけで実現できそうです。 `app/views/api/comments/_user_icons.json.jbuilder` ```ruby json.comments do json.array! comments.commented_users do |user| user.extend(UserDecorator::Role) json.user_icon user.avatar_url json.user_id user.id json.primary_role user.primary_role end end...

@komagata > extendが大丈夫なのかちょっと怖い感じがしました。 > >https://www.satoryu.com/blog/2021/08/01/how-to-use-active_decorator-for-ruby-object.html > >partial使ったときにdecorateされるのでjbuilder内でpartialを使うのはどうでしょうか？ ありがとうございます！`extend` を使用すると、オブジェクトの振る舞いがその場その場で変更されるためにコードを追うのも難しくなるのと、意図しない挙動に繋がる恐れがあるので避けたいと思います🙏 参考URLもありがとうざいます！ 現在の実装のままでいきたいと思いますがいかがでしょうか？ `app/views/api/reports/index.json.jbuilder` - api/comments/user_iconsに渡した先でuser_roleを使える現在の実装 ```ruby json.reports @reports do |report| json.partial! "api/reports/report", report: report json.partial! "api/reports/checks", checks: report.checks json.partial! "api/comments/user_icons", report:...