1-hour workshop: Include the page data owner in the filter exercise context
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 78/100
- Issue type
- Documentation
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- python
- Domain
- documentation
Research direction
Start with content/1-hour/4-add-feature.md, then read app/client/src/pages/index.astro, app/client/src/components/DogList.astro, and app/server/app.py to distinguish request, filtering, pagination, and presentation responsibilities. Validate the workshop from a fresh template copy; done means Step 4 names the page file, explains the query flow, and documents how pagination behaves when filters change.
Written by the indexing model from the issue text.
Description
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.
- Dominant language
- Python
- Stars
- 80
- Forks
- 161
- Avg merge
- 31m
- Merged PRs (30d)
- 1
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from github-samples/pets-workshop
-
priority: P0
Difficulty 2/5 1-2 days Newbie friendliness 78/100
github-samples/pets-workshop#268 · 1 comment ·
-
priority: P0
Difficulty 1/5 1-3 hours Newbie friendliness 88/100
github-samples/pets-workshop#267 · 1 comment ·
-
priority: P1
Difficulty 1/5 1-3 hours Newbie friendliness 92/100
github-samples/pets-workshop#261 · 1 comment ·
-
priority: deferred
Difficulty 4/5 3-5 days Newbie friendliness 45/100
github-samples/pets-workshop#278 · 1 comment ·
-
priority: deferred
Difficulty 4/5 3-5 days Newbie friendliness 68/100
github-samples/pets-workshop#277 · 1 comment ·
All issues in github-samples/pets-workshop
Similar issues
-
essnmx good first issue
Difficulty 1/5 Under an hour Newbie friendliness 95/100
-
[Feature] 奇物选择添加优先级 Open
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
syfoud/Simulated_Scepter#174 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Giskard-AI/giskard-oss#2840 · 1 comment ·
-
A claim comment carrying the issue number is silently declined while the workflow reports success Openarea: repo bug perceived difficulty: 2
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
yeti-platform/yeti#1380 ·