github-samples / github-samples/pets-workshop
full-day workshop: Include the page data owner in the filter exercise
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 80
- Fork
- 161
- Merge medio
- 31m
- PR unite (30g)
- 1
Descrizione
Problem
Step 6 says filtering requires the Flask backend, tests, and Astro frontend, but tells learners to supply only:
app/server/app.pyapp/server/test_app.pyapp/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.astroin 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.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia con le istruzioni del workshop Step 6 ed esamina app/server/app.py, app/server/test_app.py, app/client/src/pages/index.astro e app/client/src/components/DogList.astro. Aggiorna i requisiti prescritti relativi al contesto e al comportamento in modo che siano espliciti il filtraggio connesso all’API, la cancellazione e la gestione delle pagine, quindi esegui gli unit test aggiornati per verificare il filtraggio dell’endpoint nei casi di sola razza, sola disponibilità e combinato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- flask, python
- Ambito
- documentation, testing
- Tipo di issue
- Documentazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Attiva
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 78/100