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

ブログ投稿の文言を変更 #5464

Merged
merged 1 commit into from
Sep 5, 2022

Conversation

yuki-snow1823
Copy link
Contributor

@yuki-snow1823 yuki-snow1823 commented Aug 31, 2022

Issue

概要

管理者かメンターでログインした際のメニューの「ブログ投稿」を「ブログ記事作成」に変更。
Good First Issueとして担当させていただきました。

変更確認方法

  1. ブランチfeature/change-word-of-blog-postingsをローカルに取り込む
  2. bin/rails sでローカル環境を立ち上げる
  3. 管理者かメンターでログイン
  4. 右側のアイコンをクリックしてメニューの表示を確認してください

変更前

スクリーンショット 2022-08-31 18 04 37

変更後

スクリーンショット 2022-08-31 18 06 21

@yuki-snow1823 yuki-snow1823 marked this pull request as ready for review August 31, 2022 09:08
@yuki-snow1823
Copy link
Contributor Author

@yuma-matsui @siso25
お疲れ様です。堀越です。
初レビュー依頼させていただきました。お手隙の際によろしくお願いします☀️

Copy link
Contributor

@yuma-matsui yuma-matsui left a comment

Choose a reason for hiding this comment

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

初PRお疲れ様でした!

LGTMです🙆‍♂️

@yuki-snow1823
Copy link
Contributor Author

@yuma-matsui ありがとうございました!

@yuki-snow1823 yuki-snow1823 removed the request for review from siso25 September 1, 2022 01:43
@yuki-snow1823
Copy link
Contributor Author

@siso25
すみません、2人にご依頼するものと勘違いしておりました(__)💦
@yuma-matsui さんからApproveいただいたのでレビュー依頼外させていただきました。

@yuki-snow1823
Copy link
Contributor Author

@komagata
お疲れ様です。開発メンバーからのApproveをいただいたため、レビューお願いいたします。
本プルリクはGood First Issueとしてお任せいただいたものです。

@siso25
Copy link
Contributor

siso25 commented Sep 3, 2022

@yuki-snow1823
承知しました!返信遅くなりまして申し訳ありません🙇‍♂️

@yuki-snow1823
Copy link
Contributor Author

@siso25
とんでもありません!引き続きよろしくお願いします💪

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 32daefc into main Sep 5, 2022
@komagata komagata deleted the feature/change-word-of-blog-postings branch September 5, 2022 16:43
@github-actions github-actions bot mentioned this pull request Sep 5, 2022
20 tasks
@yuki-snow1823
Copy link
Contributor Author

ありがとうございます。
ステージングでのチェックに進みます!

@yuki-snow1823
Copy link
Contributor Author

正常な状態が確認できたのでリリースノート追記しました。

@yuki-snow1823
Copy link
Contributor Author

@komagata @machida
お疲れ様です。こちらメンター権限のログインが必要なため、本番環境で動作確認をお願いいたします。

@komagata
Copy link
Member

@yuki-snow1823 本番環境で動作することを確認しました。

@yuki-snow1823
Copy link
Contributor Author

yuki-snow1823 commented Sep 20, 2022

@komagata
ありがとうございます。完成へ移動させます。

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