镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

fix(web): link the overview's working stories to the in-progress phase - #540

Open
maruf1020 wants to merge 1 commit into
theam:mainfrom
maruf1020:fix/overview-working-stories-link
Open

maruf1020 wants to merge 1 commit into
theam:mainfrom
maruf1020:fix/overview-working-stories-link

Conversation

@maruf1020

Copy link
Copy Markdown

Problem

In the project overview's "Running now" panel, "Working stories →" links to /stories?status=working (apps/web/app/(app)/projects/[projectId]/page.tsx). The Stories page only reads phase (parseStoriesSearch in apps/web/lib/backlog-presentation.ts has no status key), and working is not a backlog phase. Clicking the link opens the default "open" view instead of the stories in progress. Every other overview link already uses ?phase=....

Fix

Link to ?phase=in_progress.

Tests

apps/web/test/overview-page.test.tsx: finds the "Working stories →" link in the Running now section, parses its query with parseStoriesSearch and expects phase to equal ["in_progress"] (fails on the old URL).

Commands run

biome check (2.5.2) on the changed files: clean. Vitest not run locally; relying on CI.

The "Working stories" link on the project overview pointed at
/stories?status=working. The Stories page only reads the `phase` search
parameter (parseStoriesSearch ignores `status`), and `working` is not a
backlog phase, so the link silently opened the default open view instead
of the stories that are in progress.

Link to ?phase=in_progress and assert in the overview test that the link
round-trips through parseStoriesSearch to that phase.

This branch has not been deployed

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant