hirano-vm4
hirano-vm4
@kurumadaisuke ## レビュー依頼 お疲れ様です🙏 お忙しい中、申し訳ありません!レビューをお願いすることは可能でしょうか🙆? 急ぎではありませんので、無理のないタイミングで大丈夫です! ご検討のほど、よろしくお願いいたします🙇♂️
@kurumadaisuke 承知しました!またの機会にお願いします🙏
@SuzukaHori ## レビュー依頼 お疲れ様です! お忙しい中、申し訳ありません!レビューをお願いすることは可能でしょうか🙆? プラクティス終盤にお願いするのは申し訳ないと感じつつも、他のプラクティス含めてレビューできる方がなかなか見つからずの依頼になります🙏 急ぎではありませんので、無理のないタイミングで大丈夫です! ご検討のほど、よろしくお願いいたします🙇♂️
@SuzukaHori ありがとうございました!非常に助かります🙏4連敗中だったのでお願いしてしまいました🙇♂️ 急がないので来週以降とか学習に支障のないタイミングでよろしくお願いします🎉 ### 追記 1点だけリファクタリングさせてもらっています([6142c7c](https://github.com/fjordllc/bootcamp/pull/7518/commits/6142c7c3907d5db032ae31a457c0faefa98f1090))🙏 CIがFLAKYなテストが通りにくくなっており、対処中です🙇♂️
@SuzukaHori お疲れ様です!無事例のFlakyなテスト改善したので、こちらもCI通りました🙏来週以降で問題ないので引き続きこちらのレビューお願いいたします〜🙆
@SuzukaHori お忙しいところ、レビュー対応いただきありがとうございます🙏 アドバイスいただいた点について修正&コメントを返していますので、無理ないタイミングで確認お願いします😊
@SuzukaHori 承知しました!ありがとうございました💪助かりました〜引き続きよろしくお願いいたします😊 @komagata こちら、チームメンバーによるレビューが終了しました!レビューをお願いいたします🙏
@machida ~~本日の開発MTGで依頼のあったバリデーション件、コードを確認したところいつでもバリデーションがかかる状況になっていました😅~~ ~~変更をかけますので、町田さんの意図が合っているか今一度大事なことなので擦り合わせさせてください🙏~~ ## ~~バリデーションの条件について~~ ~~`:profile_image(プロフィール画像)` `:profile_name(プロフィール名)` `:profile_job(職業)` `:profile_text(自己紹介文)`~~ ~~こちら4点について、現状は **「メンターであれば常に入力必須」** となっていますが、**「プロフィールを非公開のときは空欄でもOK」** という変更で大丈夫でしょうか?~~ ## 追記 上記の点について、Discordにて町田さんから方向性の修正がありました。以下の対応となりました。 ## 変更点 - バリデーションの部分は削除 - 未入力があっても、プロフィールを公開する設定になっていたら、入力されているものだけを表示(ただし、view側のif文で名前だけは最低限入力している場合のみ表示) - アイコンがない場合は画像非表示ではなく、アイコンがない場合に表示する白黒のピヨルド画像を代わりに表示する ## 理由 - バリデーションの条件を変更し「プロフィールを非公開にするときは空欄でもOKにする」にするという仕様だと、プロフィールが非表示で未入力にしたとしても、ブログ記事の著者表示を行う際に、未入力があると困る部分が結局でてしまう - 今回はバリデーションをなくすことで、デプロイ時のバタバタを避け、仕様の変更をシンプルに保つ...
@komagata こちら、駒形さんにレビュー依頼をしている途中でしたが、昨日の開発ミーティングでバリデーションの実装について町田さんから相談があり仕様の変更がありました。 [変更点](https://github.com/fjordllc/bootcamp/pull/7518#:~:text=%E3%81%A8%E3%81%AA%E3%82%8A%E3%81%BE%E3%81%97%E3%81%9F%E3%80%82-,%E5%A4%89%E6%9B%B4%E7%82%B9,-%E3%83%90%E3%83%AA%E3%83%87%E3%83%BC%E3%82%B7%E3%83%A7%E3%83%B3%E3%81%AE%E9%83%A8%E5%88%86)については、対応終了しているのでレビューを引き続きお願いいたします🙏
@komagata こちらconflictを解消しました!よろしくお願いします🙏