github-samples / github-samples/pets-workshop
1-hour workshop: Include the page data owner in the filter exercise context
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 80
- Forks
- 161
- Ø Merge
- 31 Min.
- Gemergte PRs (30 T.)
- 1
Beschreibung
Problem
Step 4 asks learners to add only app/server/app.py and app/client/src/components/DogList.astro to Copilot Edits. However, app/client/src/pages/index.astro owns the /api/dogs request, reads URL query parameters, and renders pagination. DogList.astro only receives and renders a dogs prop.
Without the page file in context, Copilot can produce a partial client-side filter or miss the request and pagination changes needed for the documented server-side filtering flow.
Suggested change
Update content/1-hour/4-add-feature.md to include app/client/src/pages/index.astro in the required context and explain at a high level which file owns the API request versus presentation.
Acceptance criteria
- Step 4 includes
app/client/src/pages/index.astroin the files supplied to Copilot. - The expected flow from filter controls to URL/API query parameters is stated.
- Existing pagination parameters are preserved, or the page is deliberately reset, when filters change.
- The exercise is validated from a fresh template copy.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit content/1-hour/4-add-feature.md und lies anschließend app/client/src/pages/index.astro, app/client/src/components/DogList.astro und app/server/app.py, um die Verantwortlichkeiten für Anfrage, Filterung, Paginierung und Darstellung zu unterscheiden. Überprüfe den Workshop anhand einer frischen Kopie der Vorlage; abgeschlossen bedeutet, dass Schritt 4 die Seitendatei nennt, den Abfragefluss erklärt und dokumentiert, wie sich die Paginierung ändert, wenn sich Filter ändern.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- documentation
- Issue-Typ
- Dokumentation
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Aktiv
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 78/100