Skip to content

Allow selection of devfile ref/path from workspace create/update Vue UI

Chad Woolley requested to merge caw-allow-devfile-path-ref-selection into master

Issue: Allow selection of devfile ref/path from worksp... (#392382 - closed)

What does this MR do and why?

Allow devfile path and ref selection

Implements Allow selection of devfile ref/path from worksp... (#392382 - closed)

Additional changes:

  • Adds a column to the Workspaces list (User and Admin versions) with a link to the workspace's devfile.
  • Add some auto-focus on fields for better usability
  • Defaults first agent in the list for better usability
  • Renames some methods and queries related to ee/app/assets/javascripts/remote_development/services/utils.js to be more appropriate
  • Removed client-side validation of devfile existence, and rely only on server. This means there's no more conditional enabling of the submit button based on devfile existence.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

New form fields and hovers

Screenshot_2024-02-17_at_1.38.31_PM

Screenshot_2024-02-17_at_1.38.58_PM

New table column

Screenshot_2024-02-21_at_10.42.01_PM

How to set up and validate locally

  • Create new workspace
  • Test both info hovers and links
  • Specify invalid devfile ref + path combinations, ensure correct error is displayed
  • Ensure default values work
  • Ensure custom devfile ref and path work
  • Verify devfile path is properly displayed in list
  • Verify proper specified non-default branch is cloned into the workspace
Edited by Chad Woolley

Merge request reports

Loading