-
-
Notifications
You must be signed in to change notification settings - Fork 4.5k
feat(issues): Display exact release in dropdown #104565
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
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.
| onSelected={(statusDetails: ResolvedStatusDetails) => | ||
| handleAnotherExistingReleaseResolution(statusDetails) | ||
| } | ||
| organization={organization} | ||
| projectSlug={projectSlug} | ||
| /> | ||
| )); |
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
There was a problem hiding this comment.
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)')} |
There was a problem hiding this comment.
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.
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
after