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

Abierto
#265 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
78/100
Tipo de issue
Documentación
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
flask, python
Área
documentation, testing

Línea de trabajo

Comienza con las instrucciones del taller de Step 6 e inspecciona app/server/app.py, app/server/test_app.py, app/client/src/pages/index.astro y app/client/src/components/DogList.astro. Actualiza los requisitos prescritos de contexto y comportamiento para que el filtrado conectado a la API, el borrado y el manejo de páginas queden explícitos; después, ejecuta las pruebas unitarias actualizadas para verificar el filtrado del endpoint en los casos de solo raza, solo disponibilidad y combinado.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.
Lenguaje dominante
Python
Estrellas
80
Forks
161
Merge medio
31 min
PR fusionados (30 d)
1

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de github-samples/pets-workshop

Todos los issues de github-samples/pets-workshop

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.