-
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
ユーザー > 日報 のページのタイトルを変更 #4061
ユーザー > 日報 のページのタイトルを変更 #4061
Conversation
@chung3011 まず他の生徒の方にレビュー依頼して、OKでしたら改めて僕にレビュー依頼する流れになっていますので、お願いします〜。 |
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.
レビューしました! LGTMです〜
@chung3011 相手が気付きづらいのでレビュー依頼する時は、メンションをつけてコメントでも依頼お願いします〜 |
@komagata 、お疲れさまです。 |
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.
@chung3011
すみません、一度レビュー通してしまったのですが、Lintとテストが通っていないようですので、そちらのご対応をお願いいたします〜🙏
@komagata
誤ってレビューリクエストを送ってしまいましたので、review ready になるまでお待ちいただけますと幸いです。よろしくお願いいたします🙏
Reopen PR |
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.
@chung3011 さん
お疲れさまです〜
こちらのプルリクエストですが、2/2(水)の振返りミーティングでの作業で、Lintとテストが通ったので、レビューさせていただきます🙏
コードと動作確認をしたところ大丈夫でした!
ブランチ名は、ブートキャンプのルールですとfeature/change-user-reports-title
なのですが、ミーティングでkomagataさんが「今回はこちらで大丈夫」と仰っていたので、OKです〜🙆♀️
ブランチ名のルールはこちらのチーム開発の作業手順に載っていますので、よろしければ参考になさってください〜
@komagataさん、お疲れさまです。 |
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.
確認しました、OKですー🙆♂️
@Saki-htr レビューOKでしたらReviewのステータスを変更お願いします〜(レビューワーに設定された人全員がOKにならないとマージできないので〜) |
@komagata |
#3705
ユーザー > 日報のページのタイトルを変更しました。
変更前
変更後
テスト