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

未关闭

#265 创建于 2026年9月14日 1 条评论 0 个 reaction 已指派 0 人在 GitHub 查看

还没有人认领这个 Issue。

评估

难度: 3/5
预计耗时: 1-2 天
新手友好度: 78/100

Issue 类型: 文档
描述清晰度: 描述清楚
活跃度: 活跃
技术栈: flask, python
领域: documentation, testing

调研方向

从 Step 6 workshop instructions 开始，检查 app/server/app.py、app/server/test_app.py、app/client/src/pages/index.astro 和 app/client/src/components/DogList.astro。更新规定的上下文和行为要求，明确通过 API 连接的筛选、清除和页面处理，然后运行更新后的单元测试，验证仅按品种、仅按可用性以及组合条件这几种情况下的 endpoint 筛选。

由索引模型根据 Issue 内容生成。

描述

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.

主要语言: Python
星标: 80
派生: 161
平均合并: 31 分钟
30 天内合并 PR: 1

贡献指南

打开贡献指南

从这里开始

先读完整个 Issue，再读项目的贡献指南。
在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
Fork 仓库，在一个分支上完成修改。
提交 Pull Request，并在描述里引用这个 Issue 编号。

github-samples/pets-workshop 的其他 Issue

full-day workshop: Add Azure prerequisites, cost warning, and resource cleanup 未关闭

priority: P0

难度 2/5 1-2 天新手友好度 78/100

github-samples/pets-workshop#268 · 1 条评论 · 2026年9月14日创建
full-day workshop: Add the missing commit step before creating the feature pull request 未关闭

priority: P0

难度 1/5 1-3 小时新手友好度 88/100

github-samples/pets-workshop#267 · 1 条评论 · 2026年9月14日创建
1-hour workshop: Add concrete validation scenarios for the filter feature 未关闭

priority: P1

难度 1/5 1-3 小时新手友好度 92/100

github-samples/pets-workshop#261 · 1 条评论 · 2026年9月14日创建
1-hour workshop: Include the page data owner in the filter exercise context 未关闭

priority: P1

难度 2/5 1-3 小时新手友好度 78/100

github-samples/pets-workshop#257 · 1 条评论 · 2026年9月14日创建
GitHub security workshop: Add security overview, governance, and rollout exercise 未关闭

priority: deferred

难度 4/5 3-5 天新手友好度 45/100

github-samples/pets-workshop#278 · 1 条评论 · 2026年9月14日创建

查看 github-samples/pets-workshop 的全部 Issue

相似的 Issue

[Bug] reef-hermes tells me to resume with hermes --resume, which does not work from my shell 未关闭

area: harness bug status: needs-triage

难度 2/5 1-3 小时新手友好度 75/100

Human-Agent-Society/reef#625 · 2026年9月24日创建
Bump transitive axios to >=1.18.0 to address GHSA-hcpx-6fm6-wx23 未关闭

难度 2/5 1-3 小时新手友好度 70/100

coinbase/cdp-sdk#828 · 2026年9月24日创建
AccessibleResourceCard/AccessibleChannelCard crash when a resource has more than one learning activity (getActivityTags returns an object instead of an array) 未关闭

难度 1/5 1 小时以内新手友好度 80/100

learningequality/kolibri#15351 · 2 条评论 · 2026年9月24日创建
audeer.path(): optionally expand environment variables 未关闭

难度 2/5 1-3 小时新手友好度 75/100

audeering/audeer#212 · 1 条评论 · 2026年9月24日创建
Name consistency 未关闭

难度 2/5 1-3 小时新手友好度 75/100

eellak/triplestore#65 · 1 条评论 · 2026年9月24日创建

更多 Python Issue