Skip to content

Conversation

@scttcper
Copy link
Member

@scttcper scttcper commented Dec 9, 2025

Refactors the resolved in release dropdown to use compact select instead of the deprecated select async component.

Pulls any exact release match to the top of the search results, this fixes a bug where if you had more than 100 releases that contained the same query it might not display the exact match.

Adds a link to open release details in a new tab.

before

image

after

image image

Refactors the resolved in release dropdown to use compact select instead of the deprecated select async component.

Pulls any exact release match to the top of the search results, this fixes a bug where if you had more than 100 releases that contained the same query it might not display the exact match.
@scttcper scttcper requested a review from a team December 9, 2025 00:26
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 9, 2025
Comment on lines 340 to 345
onSelected={(statusDetails: ResolvedStatusDetails) =>
handleAnotherExistingReleaseResolution(statusDetails)
}
organization={organization}
projectSlug={projectSlug}
/>
));

This comment was marked as outdated.

Copy link
Member Author

Choose a reason for hiding this comment

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

modal has OrganizationContext now

<Version version={release.version} anchor={false} />{' '}
{isVersionInfoSemver(release.versionInfo.version)
? t('(semver)')
: t('(non-semver)')}
Copy link
Contributor

Choose a reason for hiding this comment

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

Bug: Missing null check for versionInfo causes crash

The code accesses release.versionInfo.version directly without optional chaining, but uses release.versionInfo?.package with optional chaining on lines 35 and 44, indicating versionInfo may be undefined. Other parts of the codebase (e.g., useProjectReleaseVersionIsSemver.tsx) explicitly check for !data?.versionInfo before accessing .version. If versionInfo is undefined, isVersionInfoSemver(release.versionInfo.version) will throw a runtime error.

Fix in Cursor Fix in Web

@scttcper scttcper merged commit 04e8ff1 into master Dec 9, 2025
48 checks passed
@scttcper scttcper deleted the scttcper/resolved-in-release-dropdown branch December 9, 2025 23:58
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.

3 participants