full-day workshop: Include the page data owner in the filter exercise

オープン
#265 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
3/5
見積もり時間
1〜2日
初心者へのやさしさ
78/100
issue の種類
ドキュメント
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
flask, python

調査の方向性

Step 6 のワークショップ手順から始め、app/server/app.py、app/server/test_app.py、app/client/src/pages/index.astro、app/client/src/components/DogList.astro を調べます。API に接続したフィルタリング、クリア、ページ処理が明示されるように、規定されたコンテキストおよび動作要件を更新し、その後、更新した unit test を実行して、犬種のみ、利用可能性のみ、両方を組み合わせたケースで endpoint のフィルタリングを検証します。

索引モデルが issue の本文から書いたものです。

説明

priority: P1

Problem

Step 6 says filtering requires the Flask backend, tests, and Astro frontend, but tells learners to supply only:

  • app/server/app.py
  • app/server/test_app.py
  • app/client/src/components/DogList.astro

app/client/src/pages/index.astro owns the /api/dogs request, reads URL query parameters, and renders pagination. DogList.astro only renders the dogs prop. Omitting the page owner encourages a partial implementation that cannot reliably connect controls to server-side query parameters or preserve pagination behavior.

Suggested change

Include index.astro in the prescribed context and add implementation-agnostic behavior checks.

Acceptance criteria

  • Step 6 includes app/client/src/pages/index.astro in Copilot context.
  • The prompt requirements describe how control changes reach the API.
  • Breed-only, availability-only, and combined filtering are validated.
  • Clearing filters restores the full list.
  • Filter changes handle the current page deliberately, such as resetting to page 1.
  • Updated unit tests prove endpoint filtering rather than only the original response shape.
主要言語
Python
スター
80
フォーク
161
平均マージ
31分
マージ済み PR(30日)
1

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

github-samples/pets-workshop のほかの issue

github-samples/pets-workshop の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。