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

Markdown in displayName #3804

Open
say5 opened this issue Nov 29, 2024 · 3 comments
Open

Markdown in displayName #3804

say5 opened this issue Nov 29, 2024 · 3 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@say5
Copy link

say5 commented Nov 29, 2024

Feature request

Support Markdown in displayName field.

Use case

Markdown in displayName would improve UX of dashboard, for example it would be possible to specify link to PR, commit, highlight some information, etc.
Good example of it can be found in Argo Workflows.

@say5 say5 added the kind/feature Categorizes issue or PR as related to a new feature. label Nov 29, 2024
@AlanGreene
Copy link
Member

Thanks for the feature request @say5. That's something we may consider as part of #2306

I'm not sure displayName is the correct place to include URLs or other potentially large values, that seems better suited to a description field or similar. However, this is something we can explore as part of the redesign of the PipelineRun and TaskRun details pages.

Apart from the examples in the Argo docs you provided above, do you have concrete examples / use cases for how you plan to use such a feature? Having specific use cases really helps to understand the problem and ensure we provide the correct solution.

@say5
Copy link
Author

say5 commented Nov 29, 2024

@AlanGreene in our use case we mostly trigger PipelineRuns from GitHub, in other CI/CD systems there is a page with all builds you have with basic primary information in one place - author on changes, link to PR, branch name, etc. Currently it is not possible with Tekton/Dashboard PipelineRuns page (no displayName in PipelineRun). We have displayName in pipelinerun.tasks so we are using it to put all this info to the first task in PipelineRun, far from ideal but better than nothing, still there is no way to make links work, also in UI it shows rather limited string.

Here is an example of similar issue: #3323

Regarding task.description - afaik there is no variable substitution for it, as it is just general description of a task and idea is to use substitution to display dynamic information about "build". In my view two changes can improve UX a lot:

  1. displayName for PipelineRun.
  2. Markdown support to allow links, size, style, emojis, etc.

@AlanGreene
Copy link
Member

Regarding task.description - afaik there is no variable substitution for it, as it is just general description of a task

That is correct. I wasn't referring to that field specifically, rather pointing out that a field to provide a longer form description may be more suitable for some use cases than the display name, which ideally should be kept relatively short.

Thanks for the additional detail, it's very helpful. I think this would make sense to consider after PipelineRun displayName is supported in Tekton Pipelines.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

No branches or pull requests

2 participants