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

プラクティス個別ページの終了条件を修了条件にするためにlocalesの日本語訳を変更 #8089

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ayu-0505
Copy link
Contributor

@ayu-0505 ayu-0505 commented Sep 25, 2024

Issue

概要

プラクティス個別ページ下部の修了条件が記載されている箇所において、
「終了条件」となっていた文言を「修了条件」に変更します。

スクリーンショット 2024-09-19 15 18 42

変更確認方法

  1. feature/change_text_of_practice_goalをローカルに取り込む
    1. git fetch origin pull/8089/head:feature/change_text_of_practice_goal
    2. git checkout feature/change_text_of_practice_goal
  2. foreman start -f Procfile.devでローカルサーバーを立ち上げる
  3. 任意のアカウントでログイン
  4. プラクティス個別ページを開き、下部にある修了条件を確認

Screenshot

変更前

スクリーンショット 2024-09-19 15 18 23

変更後

スクリーンショット 2024-09-20 16 14 09

@ayu-0505 ayu-0505 self-assigned this Sep 25, 2024
@ayu-0505 ayu-0505 changed the title localesのja.ymlにおいてpracticeのgoalを修了条件に変更 プラクティス個別ページの終了条件を修了条件にするためにlocalesの日本語訳を変更 Sep 25, 2024
@ayu-0505 ayu-0505 marked this pull request as ready for review September 25, 2024 05:24
@ayu-0505
Copy link
Contributor Author

@mousu-a さん、お疲れ様です🍵
初レビュー依頼させていただきます。
お忙しいところすみませんがまた確認をお願いいたします🙏

@mousu-a
Copy link
Contributor

mousu-a commented Sep 25, 2024

@ayu-0505さん
レビュー了解です!
明日中には返せると思いますのでお待ちくださいー🙏

@mousu-a
Copy link
Contributor

mousu-a commented Sep 25, 2024

@ayu-0505
お疲れ様です!
早速ですがレビューOKです!🙆

本筋とは関係ないですが変更確認方法をこんな感じにするとレビュワーが助かるかもです!

  1. feature/change_text_of_practice_goalをローカルに取り込む
    1.  git fetch origin pull/8089/head:feature/change_text_of_practice_goal
    2.  git checkout feature/change_text_of_practice_goal

(コピペするために文字の先頭に空白を入れています)

ついでに、「任意のプラクティス個別ページを開き、下部にある修了条件を確認」も「プラクティス個別ページを開き、下部にある修了条件を確認」みたいにしちゃっても良いかもです!良ければお試しください🙇‍♂️
(ご存知だったら申し訳ないですが、文字を範囲選択しURLをペーストするとこんな風にリンクを作れます)

こちらからはApproveとさせていただきます〜!
初レビュー依頼ありがとうございました😄

@ayu-0505
Copy link
Contributor Author

ayu-0505 commented Sep 26, 2024

@mousu-aさん、確認いただきありがとうございます!

Descriptionは他の方のを参考に見よう見まねで書いたのですが、
mousuさんの書き方が確かにレビュワー対して親切でとても参考になりました👀
とくにgit fetch origin pull/8089/head:~という書き方を知らなかったので勉強になりました。
また、リンクを貼っておくという発想もなかったので「なるほど〜!」と思いました。
早速今回のPRも修正し、次回以降にも活かしていきたいと思います💪

Approveありがとうございます!
色々教えていただき感謝いたします🙏

@ayu-0505
Copy link
Contributor Author

@komagataさん、お疲れ様です🍵
メンバーレビューでOKいただけたので確認をお願いいたします。

@ayu-0505
Copy link
Contributor Author

ayu-0505 commented Sep 30, 2024

@mousu-aさん、たびたびすみません!
Approveですが、こちらのPRページの右上のReviewers部分の名前横のマークが緑チェックに変わる、というのを教えていただきました。
お手数ですが、またお時間ある時で構わないので確認をお願いいたします🙏

Copy link
Contributor

@mousu-a mousu-a left a comment

Choose a reason for hiding this comment

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

@ayu-0505
すみません、うっかりしていました😭
今度こそ、Approveとさせていただきます!

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.

2 participants