Skip to content
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

お知らせ個別ページ作成ボタンの色を変更#2842 #2986

Merged

Conversation

makiichikawa
Copy link
Contributor

@makiichikawa makiichikawa commented Jul 15, 2021

ref #2842
標記の件で確認よろしくお願いいたします🙇‍♀️

変更前

今週のリリース2021年7月15日___FJORD_BOOT_CAMP(フィヨルドブートキャンプ)

変更後

テストのお知らせ___FJORD_BOOT_CAMP(フィヨルドブートキャンプ)

@machida
Copy link
Member

machida commented Jul 15, 2021

@makiichikawa

Pull Request にレビューを頼むときは、Draft の状態ではできません。Ready for review をクリックしてレビューできる状態にしてくださいー

続いて、受講生の誰かを指名お願いします。今回は yana-gi さんにお願いしてみましょう。右上のレビュワーとのところから yana-gi さんを選択し、コメントから yana-gi さんにメンションを付けて、レビューお願いします的なメッセージをお願いします。

@makiichikawa makiichikawa requested a review from yana-gi July 15, 2021 11:16
@makiichikawa makiichikawa marked this pull request as ready for review July 15, 2021 11:20
@makiichikawa
Copy link
Contributor Author

@machida
ご教授ありがとうございます🙇🏼‍♂️
yana-giさんにレビューをお願いしてみます。

@yana-gi
大変恐縮ですが、レビューよろしくお願いいたします。

Copy link
Contributor

@yana-gi yana-gi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

レビュー全然恐縮じゃないですよ〜
チームでやるものなのでこれからはフランクにお願いしちゃってください😄

見ました!
内容も大丈夫そうですし、PRコメントも記載されていてナイスです👍

1点だけ修正して欲しいのですが、PRコメントに ref #2842 のようにissue番号も記載しておいてください〜
それ以外は大丈夫そうなのでapproveしておきます!

@makiichikawa makiichikawa requested a review from komagata July 17, 2021 09:38
@makiichikawa
Copy link
Contributor Author

@yana-gi
レビューありがとうございました🙏
ご指摘いただいた、PRリクエストのコメントについて、当issueが参照できるように修正しました。
次はフランクに依頼してみようと思います👍

@komagata
こちらのレビューお願いいたします🙏

Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確認しました、OKですー🙆‍♂

@komagata komagata merged commit 67cba93 into main Jul 19, 2021
@komagata komagata deleted the feature/change-color-of-button-to-create-announcement-page branch July 19, 2021 06:43
@github-actions github-actions bot mentioned this pull request Jul 19, 2021
20 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants