Skip to content

Conversation

@krjakbrjak
Copy link

Adds a dynamic dropdown for repository owners in both GiteaSCMSource and GiteaSCMNavigator configuration forms, improving user experience when configuring Gitea repositories.

  1. doFillRepoOwnerItems method:
    • fetches all accessible repositories using c.fetchCurrentUserRepositories()
    • extracts unique owner names from the repositories
    • returns them as a ListBoxModel for the dropdown
    • maintains the current selection if it's valid
  2. owner field in the UI is changed from textbox to select with the right dependencies.

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

@krjakbrjak krjakbrjak force-pushed the VNI-discover-repos-improvements branch from 499ce65 to b1a4e4a Compare November 9, 2025 16:43
@krjakbrjak krjakbrjak force-pushed the VNI-discover-repos-improvements branch from b1a4e4a to bbf7c64 Compare November 9, 2025 20:04
@lafriks
Copy link
Contributor

lafriks commented Nov 9, 2025

Linter still fails ;)

1. `doFillRepoOwnerItems` method:
   * fetches all accessible repositories using `c.fetchCurrentUserRepositories()`
   * extracts unique owner names from the repositories
   * returns them as a ListBoxModel for the dropdown
   * maintains the current selection if it's valid
2. `owner` field in the UI is changed from `textbox` to `select` with the right dependencies.

Signed-off-by: Nikita Vakula <[email protected]>
@krjakbrjak krjakbrjak force-pushed the VNI-discover-repos-improvements branch from bbf7c64 to e44af0a Compare November 9, 2025 20:40
@krjakbrjak
Copy link
Author

Linter still fails ;)

fixed

@krjakbrjak
Copy link
Author

@lafriks I fixed the one warning reported.
Let me know if this looks useful for the plugin or if anything needs adjusting. Thanks! 🙌

@lafriks lafriks requested a review from justusbunsi November 20, 2025 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants