-
Notifications
You must be signed in to change notification settings - Fork 116
fix: Show repo owner in builds repo selector #5417
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
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.
Pull Request Overview
This PR fixes the repository selector in the builds page to display repository owners alongside repository names. The changes improve user experience by showing the full repository path (owner/repo) in the dropdown selector while maintaining the repository name as the actual value.
Key Changes:
- Sorts repositories alphabetically by name
- Updates the datalist options to display the full repository path including owner
7e431d9
to
9130f72
Compare
9130f72
to
aee8365
Compare
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.
Pull Request Overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
aee8365
to
f83c452
Compare
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.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
ccbdb7b
to
2d9863b
Compare
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.
Pull Request Overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
LGTM, thank you!
Done
Adds the user or organisation name to the list of repo suggestions to avoid confusion over the repo owner when there are two repos with the same name.
How to QA
Testing
Issue / Card
Fixes https://warthogs.atlassian.net/browse/WD-27256
Screenshots