Skip to content
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

feat: fetch available versions into dropdown when adding frozen version #96

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

johannrichard
Copy link

@johannrichard johannrichard commented Feb 16, 2025

This is a user-experience enhancement.

  • Versions are presented as a dropdown (live-updated from GitHub releases) when a valid repository is entered
  • Whenever the user presses "enter", or leaves the input field, we validate the repository and fetch the available versions (indicating pre-releases in the dropdown)
  • The "Add Plugin" button is only enabled when a valid repository/version combo is selected

Open question: we indicate a valid repository by drawing a green border around the address, and red for invalid repositories. We also enable/disable the version dropdown respectively. Not sure this is good practice, but certainly helps the user when navigating the UI.

Ideas for further enhancement:

  • validate the repository/release for the presence of a "manifest.json" file

if a valid repository is entered, we fetch a list of available versions from the repository when "enter" is pressed and let the user select from that list. Added some UI anehancements (e.g. disabled submit button if no version is selected).
@johannrichard
Copy link
Author

@TfTHacker I was wondering if you have noticed the PRs I submitted and if so, if there's anything I could do to facilitate your review?

@TfTHacker
Copy link
Owner

Johan, sorry for the delayed. I have been rather swamped these days. Can you contact me on Discord?

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