Skip to content

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

Open

Description

@softchris

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.

Activity

  1. scubaninja commented on Sep 18, 2026

    @scubaninja
    Collaborator

    Backlog review priority: P1. Tagging @scubaninja for visibility and follow-up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    priority: P1High: core correctness or reliability

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions