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

Add a webview to generate a devfile without ansible-creator #1681

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

alisonlhart
Copy link
Contributor

@alisonlhart alisonlhart commented Nov 26, 2024

This PR adds a new webview for generating a devfile from a template, designed for Red Hat OpenShift Dev Spaces.

New Create Devfile Webview

Adds a view where devfiles can be generated. The options are: Destination Path, Ansible Project Name, Container Image, and Override.

  • Destination Path: defaults to the first folder open in the current workspace, or an empty path. If the path is empty, a path must be entered into the text box before the devfile can be generated.
  • Ansible Project Name: Ansible project the devfile is being added to
  • Container Image: Image to use in the devfile
  • Override: If a file named devfile.yaml already exists at the destination path, create a new one with the form details.

Logging

Adds log output to the webview form showing that the devfile was created or that an error occurred.

Open file options

Shows a toast notification if the devfile was created successfully, and highlights an option to open it in the editor.

image

image

@alisonlhart alisonlhart marked this pull request as ready for review December 2, 2024 16:21
@alisonlhart alisonlhart requested review from a team as code owners December 2, 2024 16:21
@alisonlhart alisonlhart changed the title [DRAFT] Add a webview to generate a devfile without ansible-creator Add a webview to generate a devfile without ansible-creator Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

1 participant