Skip to content

Conversation

steverydz
Copy link
Contributor

@steverydz steverydz commented Oct 13, 2025

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

  • Run locally (builds don't work on demos)
  • Go to /<snap_name>/builds
  • In the "Select a repository" field in the autocomplete, the org/username for each snap should appear under the repo name

Testing

  • This PR has tests
  • No testing required (explain why):

Issue / Card

Fixes https://warthogs.atlassian.net/browse/WD-27256

Screenshots

example

@Copilot Copilot AI review requested due to automatic review settings October 13, 2025 13:41
@webteam-app
Copy link

Copy link
Contributor

@Copilot Copilot AI left a 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

@steverydz steverydz force-pushed the WD-27256-fix-duplicate-repos-in-builds branch from 7e431d9 to 9130f72 Compare October 13, 2025 13:51
@edisile edisile self-requested a review October 13, 2025 14:45
@steverydz steverydz force-pushed the WD-27256-fix-duplicate-repos-in-builds branch from 9130f72 to aee8365 Compare October 14, 2025 10:16
@steverydz steverydz requested a review from Copilot October 14, 2025 10:17
Copy link
Contributor

@Copilot Copilot AI left a 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>
@steverydz steverydz force-pushed the WD-27256-fix-duplicate-repos-in-builds branch from aee8365 to f83c452 Compare October 14, 2025 11:03
@steverydz steverydz requested a review from Copilot October 14, 2025 11:04
Copy link
Contributor

@Copilot Copilot AI left a 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.

steverydz and others added 3 commits October 14, 2025 12:07
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@steverydz steverydz force-pushed the WD-27256-fix-duplicate-repos-in-builds branch from ccbdb7b to 2d9863b Compare October 14, 2025 14:08
@steverydz steverydz requested a review from Copilot October 14, 2025 14:09
Copy link
Contributor

@Copilot Copilot AI left a 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.

steverydz and others added 2 commits October 14, 2025 15:10
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

@edisile edisile left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

@steverydz steverydz merged commit c99ce5d into main Oct 17, 2025
13 checks passed
@steverydz steverydz deleted the WD-27256-fix-duplicate-repos-in-builds branch October 17, 2025 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants