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

「終了条件」を「修了条件」に修正 #8363

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

Conversation

riq0h
Copy link

@riq0h riq0h commented Feb 21, 2025

Issue

#8358

概要

提出物が不要なプラクティスにおいて、ページ上部のステータス欄直下の文字列に含まれていた誤字を修正した。

変更確認方法

1.リモートリポジトリをローカルに取り込む。
git fetch origin chore/replace-end-to-mastering
git checkout chore/replace-end-to-mastering

2.ローカル環境を起動してログインを行う。
foreman start -f Procfile.devローカル環境を立ち上げる。
ユーザー名 komagata パスワード testtest でログインする。

3.問題の箇所が修正されていることを確かめる。
PC性能の見方を知るにアクセスする。(提出物が不要なプラクティスならどこでも可)
「このプラクティスに提出物はありません。終了条件をクリアしたら修了にしてください。」が「このプラクティスに提出物はありません。修了条件をクリアしたら修了にしてください。」に修正されていることを確認する。

Screenshot

変更前

2025-02-21_16-31

変更後

2025-02-21_16-23

@riq0h riq0h self-assigned this Mar 4, 2025
@riq0h riq0h marked this pull request as draft March 4, 2025 08:42
@riq0h riq0h marked this pull request as ready for review March 4, 2025 08:43
@riq0h riq0h marked this pull request as draft March 4, 2025 08:43
@riq0h riq0h marked this pull request as ready for review March 5, 2025 04:11
@riq0h
Copy link
Author

riq0h commented Mar 5, 2025

@harada-webdev
お疲れ様です! Good First Issueのレビューをお願いしたいです🙇

@harada-webdev harada-webdev self-requested a review March 5, 2025 14:15
Copy link
Contributor

@harada-webdev harada-webdev left a comment

Choose a reason for hiding this comment

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

@riq0h
Reviewersが自分に指定されていなかったので、それだけ今後確認をお願いいたします🙏
内容はOKですので、Approveしました!

@riq0h riq0h requested a review from komagata March 5, 2025 14:21
@riq0h
Copy link
Author

riq0h commented Mar 5, 2025

ありがとうございます! 以後気をつけます🙇
@komagata
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