choco
choco
@ai-24 お忙しいところ確認ありがとうございました🙇♂️🙇♂️🙇♂️
@komagata お疲れ様です! ai-24さんよりApprove頂きましたのでレビューよろしくお願い致します🙇♂️🙇♂️🙇♂️
@komagata conflict修正しましたのでご確認お願いします🙇♂️
本番環境で確認できたため、closeします!
@machida 日報個別ページというのは下記の認識であっていますでしょうか? > メンターがチェックする場合に、何か企業にも共有しておいた方がいいことが日報に含まれていた場合に、どの企業所属の人なのかがすぐわかるようにしたいから。 とあったので、adminユーザーからみれる下記のページなのかなと個人的に思ったので、 どちらの認識なのか教えて頂きたいです。
@machida 承知しました〜!
@machida 現状、研修生のみ個別日報のタイトル付近に企業ロゴを表示できるようになったんですが、テストをどうするべきか悩んでいます。過去のPRなどをみてみたところ、画像を表示させるテストが見当たらなかったのですが(自分が探した限り😓)、今回の機能追加もテストは必要なのでしょうか? 個人的にはテストは必要だと思うのですが、テストケースを作る、作らないの境目がわからず...
@machida @komagata ユーザーアイコンに関するテストを見つけたので、そこを参考にテストを作成してみようと思います。
本番環境で確認できましたのでcloseします。
本番環境で確認できましたのでcloseします。