Skip to content

fix(site): do not render invalid task status URI, fix GitHub new links - #20858

Merged
mafredri merged 1 commit into
mainfrom
frontend-task-link-bug
Nov 21, 2025
Merged

mafredri merged 1 commit into
mainfrom
frontend-task-link-bug

Conversation

@mafredri

Copy link
Copy Markdown
Member

Fixes #20429

? `${org}/${repo}#${number}`
: `${org}/${repo} pull request`;
label =
number === "new"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Drive-by fix: Handle new. I personally ran into this where open a new PR was called coder/coder#new, very confusing.

@mafredri
mafredri force-pushed the frontend-task-link-bug branch from d27401f to 8d14447 Compare November 21, 2025 12:15

@ssncferreira ssncferreira left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚗

@mafredri
mafredri merged commit e0a32e0 into main Nov 21, 2025
26 checks passed
@mafredri
mafredri deleted the frontend-task-link-bug branch November 21, 2025 12:24
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

tasks: random empty link when still starting up

3 participants