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: Show that a pipeline is used to create a template (HEXA-1167 ) #1032

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

Conversation

YolanFery
Copy link
Contributor

We want to show that a pipeline is used as the source for a particular template

Changes

  • Add a property with the source template
  • Add a badge with the source template version

Note : unclickable text will be replaced by links to the template view page once it is implemented (https://bluesquare.atlassian.net/browse/HEXA-1130)

Screenshots / screencast

Screenshot 2025-01-22 at 16 12 23 Screenshot 2025-01-22 at 16 12 31

@YolanFery YolanFery requested a review from qgerome January 22, 2025 15:22
…plement-an-elegant-way-to-show-that-a-pipeline-is-used-to-create-a-template
@qgerome
Copy link
Member

qgerome commented Jan 28, 2025

@YolanFery Could you fix the build please ?

…plement-an-elegant-way-to-show-that-a-pipeline-is-used-to-create-a-template
@YolanFery
Copy link
Contributor Author

@YolanFery Could you fix the build please ?

I just rebased, the issue was because of the tailwind upgrade

<TextProperty
id="template"
accessor={"template.name"}
label={t("Source of template")}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd drop the "Source of" and only keep "Template"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be confusing with the Source template but let's do that for now :)

…plement-an-elegant-way-to-show-that-a-pipeline-is-used-to-create-a-template
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