github-samples / github-samples/pets-workshop

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

Aperta
#265 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

priority: P1
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.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.

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. 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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.