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

ayu-0505 commented 2 months ago

Issue

プラクティス個別ページの終了条件を修了条件に変更。 #8078

概要

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

変更確認方法

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
foreman start -f Procfile.devでローカルサーバーを立ち上げる
任意のアカウントでログイン
プラクティス個別ページを開き、下部にある修了条件を確認

Screenshot

変更前

変更後

ayu-0505 commented 2 months ago

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

mousu-a commented 2 months ago

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

mousu-a commented 2 months ago

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

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

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 commented 2 months ago

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

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

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

ayu-0505 commented 2 months ago

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

ayu-0505 commented 1 month ago

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

fjordllc / bootcamp