Skip to content

fix(discover): Link issue event ids directly#116507

Merged
scttcper merged 1 commit into
masterfrom
scttcper/discover-event-links-master
May 29, 2026
Merged

fix(discover): Link issue event ids directly#116507
scttcper merged 1 commit into
masterfrom
scttcper/discover-event-links-master

Conversation

@scttcper
Copy link
Copy Markdown
Member

@scttcper scttcper commented May 29, 2026

Recreates #116485 against master. The previous PR was accidentally based on another dev branch.

Discover was still routing issue event id clicks through the project-event redirect, even when the row already had issue.id. That made some issue-event urls messy and left the redirect to clean things up later.

This requests issue.id with the hidden link context and builds the issue event url directly. Keeps the old project redirect fallback for rows that do not have issue context.

Discover was still routing issue event id clicks through the
project-event redirect, even when the row already had `issue.id`. That
made some issue-event urls messy and left the redirect to clean things
up later.

This requests `issue.id` with the hidden link context and builds
`/issues/:issueId/events/:eventId/` directly. Keeps the old project
redirect fallback for rows that do not have issue context.
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 29, 2026
@scttcper scttcper marked this pull request as ready for review May 29, 2026 18:41
@scttcper scttcper requested a review from a team as a code owner May 29, 2026 18:41
@github-actions
Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.59%

@scttcper scttcper merged commit 35f5dbc into master May 29, 2026
78 checks passed
@scttcper scttcper deleted the scttcper/discover-event-links-master branch May 29, 2026 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants