Skip to content

feat(issues): Link commit activities to pull requests - #119373

Merged
scttcper merged 7 commits into
masterfrom
scttcper/resolved-release-pr-link
Jul 13, 2026
Merged

feat(issues): Link commit activities to pull requests#119373
scttcper merged 7 commits into
masterfrom
scttcper/resolved-release-pr-link

Conversation

@scttcper

@scttcper scttcper commented Jul 9, 2026

Copy link
Copy Markdown
Member

Commit-backed issue activities already have pull request data, but the new issue activity feed mostly stops at the release or commit.

Behind issue-activity-feed-v2, this prefers the pull request for resolved-in-release activities and adds the pull request alongside referenced commits. The legacy activity feed is unchanged.

Depends on #119372

Screenshot 2026-07-13 at 3 53 05 PM

part of https://linear.app/getsentry/issue/ID-1679

Resolved-in-release activities only linked to the release even when Sentry knew which commit caused the resolution.

Prefer the associated pull request in both activity feeds and fall back to the exact commit when there is no PR. Activities without the new backend data keep the existing display.

Co-Authored-By: GPT-5 <noreply@openai.com>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jul 9, 2026
Referenced commit activities already include the associated pull request, but the activity feed only showed the commit.

Keep the commit context and add the pull request link when it is available in both activity feed variants.

Co-Authored-By: GPT-5 <noreply@openai.com>
@scttcper scttcper changed the title feat(issues): Link release resolutions to pull requests feat(issues): Link commit activities to pull requests Jul 9, 2026
Keep the pull request links limited to the new issue activity feed behind issue-activity-feed-v2. Restore the legacy renderer and remove its new test coverage.

Co-Authored-By: GPT-5 <noreply@openai.com>
scttcper and others added 2 commits July 10, 2026 09:12
The semver and non-semver suffixes add noise next to the release link in the new activity feed. Keep the release context and drop the classification text.

Co-Authored-By: GPT-5 <noreply@openai.com>
Keep the activity tests focused on the links we render instead of asserting removed text stays absent.

Co-Authored-By: GPT-5 <noreply@openai.com>
Move release and integration detail rendering out of the compact activity switch and give integration activity data an explicit shape.\n\nCo-Authored-By: GPT-5 <noreply@openai.com>
@scttcper
scttcper marked this pull request as ready for review July 13, 2026 16:39
@scttcper
scttcper requested a review from a team as a code owner July 13, 2026 16:39
@linear-code

linear-code Bot commented Jul 13, 2026

Copy link
Copy Markdown

ID-1679

@scttcper
scttcper merged commit 104c669 into master Jul 13, 2026
81 checks passed
@scttcper
scttcper deleted the scttcper/resolved-release-pr-link branch July 13, 2026 22:54
scttcper added a commit that referenced this pull request Jul 13, 2026
Keep the pull request links from #119373 while preserving the shorter sentence-style activity copy and natural commit detail order.

Co-Authored-By: Codex <noreply@openai.com>
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