Skip to content

Conversation

axel7083
Copy link
Collaborator

@axel7083 axel7083 commented Apr 9, 2025

Description

On the Quadlet Lists, add Quadlets requires as children.

Notes

  • Disabling the selection of the children to avoid deletion issue
  • Quadlet can appear several time (one at top level, then in the children row of any quadlet depending on it.)
  • Different icon is used for children (dependant quadlet) to differentiate them)

Screenshots

image

Related issues

Require rebase after

Fixes #483

Testing

  • unit tests has been provided
  1. Generate an image quadlet (e.g. nginx.image)
  2. Generate a container quadlet (e.g. nginx.container)
  3. Edit the container quadlet and change the image=nginx to image=nginx.image (must match the image quadlet you created at step 1).
  4. assert Quadlet List page properly display the image as a child of the container

Signed-off-by: axel7083 <42176370+axel7083@users.noreply.github.com>
@axel7083 axel7083 force-pushed the feature/quadlet-require/full branch from 4f61432 to 162dcea Compare April 11, 2025 09:50
@axel7083 axel7083 marked this pull request as ready for review April 11, 2025 12:11
@axel7083 axel7083 requested a review from a team as a code owner April 11, 2025 12:11
@axel7083 axel7083 requested review from benoitf, cdrage and dgolovin and removed request for a team April 11, 2025 12:11
@axel7083 axel7083 marked this pull request as draft April 15, 2025 12:50
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.

1 participant