toke

@komagata お疲れ様です。 こちらのissueなのですが、本番での動作確認がメンターじゃないと出来ないので、お願いをしてもよろしいでしょうか？ https://github.com/fjordllc/bootcamp/pull/6549

@komagata > 確認すべき画面 こちら３種類になります🙇‍♂️ https://bootcamp.fjord.jp/users https://bootcamp.fjord.jp/questions?target=not_solved https://bootcamp.fjord.jp/pages > 動作 こちらのプルリクの説明で、動画付きでデモンストレーションを用意しています🙇‍♂️ https://github.com/fjordllc/bootcamp/pull/6549

@komagata ご確認ありがとうございました🙇‍♂️} 確認が取れましたので、本issueをclose致します🙇‍♂️

本番環境での動作確認をし、okだったのでクローズします。

本番環境で動作確認できたのでcloseします

> @syo-tokeshi お疲れさまです！ こちらのレビューをお願いしたいのですが、ご都合はいかがでしょうか？ @wata00913 私でよければ全然大丈夫です😊 なるべく明後日までにお返事する形でよろしいでしょうか？ 調査して、ちゃんとレビューするようにします😊 一緒に勉強する気持ちでやります！🙇‍♂️

@wata00913 お疲れ様です。 レビュー致しました😊 レビューをする過程でたくさん勉強になりました🙇‍♂️ 何か疑問点等ございましたら、お申し付けください🙇‍♂️

@wata00913 お疲れ様です。 返信致しました🙇‍♂️ `fit? `を`just_fit?`に変えた方が良いかなー、と思う所以外は、 全て良いと思います。 レビューをする中で、圧倒的にwataさんの方が考えたのだろうな、というのが分かり、 本当に勉強になりました🙇‍♂️ ありがとうございます🙇‍♂️ こうやってレビュアーと会話する中で、命名一つ取っても、 ちゃんと考えて実装されているかが分かりました😊 > 暫くの間、コメントへの返信が遅れるかもしれないです。申し訳ありません。🙇‍♂️ かしこまりました！ 全然大丈夫ですよ〜😆

コメント致しました🙇‍♂️ 今回議論したいのはこちらになります🙇‍♂️ https://github.com/fjordllc/bootcamp/pull/6483#discussion_r1234688184 今回のケースでは、可読性を考慮して前回のコード(`assert_equal({ width: 1200, height: 630 }`)で良い気がしました🙇‍♂️ ベタ書きを採用しても良いような気がします。

@wata00913 確認いたしました！！🙇‍♂️ 全て確認の上goodと思ったので、approveします😄