Skip to content

「プロジェクト:ToDoアプリ」の節の問題のステップ4を削除#938

Open
chvmvd wants to merge 1 commit intomainfrom
fix-todo-project
Open

「プロジェクト:ToDoアプリ」の節の問題のステップ4を削除#938
chvmvd wants to merge 1 commit intomainfrom
fix-todo-project

Conversation

@chvmvd
Copy link
Contributor

@chvmvd chvmvd commented Mar 8, 2026

次のような理由から「プロジェクト:ToDoアプリ」の節の問題のステップ4を削除しました。

  • 入力欄が空である場合にToDoを追加できなくすることがそれほど重要なことではない
  • 教材で扱っていないnullを使用している
  • 内容が過度に複雑になってしまう

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

「プロジェクト: ToDoアプリ」教材から、発展ステップとして用意されていた「空のToDoを追加できないようにする(ステップ4)」を削除し、手順・サンプル構成を3ステップに整理するPRです。

Changes:

  • 「ルール」から「入力欄が空欄だとToDoの追加ができなくなります。」を削除
  • 手順説明を「4つ」→「3つ」に更新し、ステップ4の記載を削除
  • ステップ4用のサンプル(_samples/todo)を削除

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
docs/2-browser-apps/06-project/index.mdx ステップ4(発展)の説明・Answerブロックを削除し、手順全体を3ステップに更新
docs/2-browser-apps/06-project/_samples/todo/script.js ステップ4の完成形サンプルを削除
docs/2-browser-apps/06-project/_samples/todo/index.html ステップ4の完成形サンプルを削除

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

@chvmvd
Copy link
Contributor Author

chvmvd commented Mar 8, 2026

@codex review

@chvmvd chvmvd marked this pull request as ready for review March 8, 2026 06:20
@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. More of your lovely PRs please.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

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