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: Display title and description in archived view #10796

Conversation

jmeridth
Copy link
Member

Already working in normal workflow view via #9805

  • create shared react component for workflow row name to dry up usage of this code in both normal and archive workflow lists
  • add documentation regarding title and description annotations

Screenshot coming soon

Fixes #10771

@jmeridth jmeridth self-assigned this Mar 31, 2023
@jmeridth jmeridth force-pushed the jm/title-and-description-annotations-in-archive-view branch 3 times, most recently from e193bbd to e2a3416 Compare March 31, 2023 18:47
@jmeridth jmeridth marked this pull request as draft March 31, 2023 18:47
@jmeridth
Copy link
Member Author

Converting to draft because this isn't working yet. The archive workflow service does not have much in regards to annotations. UI is unchanged until the annotation can get to the new react component. Screen shot also once I get this working locally. Going to look at adding a test also.

@jmeridth jmeridth force-pushed the jm/title-and-description-annotations-in-archive-view branch from e2a3416 to 94dbaf6 Compare March 31, 2023 18:54
Already working in normal workflow view via argoproj#9805

- create shared react component for workflow row name to dry up usage of this code in both normal and archive workflow lists
- add documentation regarding title and description annotations

Signed-off-by: jmeridth <[email protected]>
@jmeridth jmeridth force-pushed the jm/title-and-description-annotations-in-archive-view branch from 94dbaf6 to e9cabe3 Compare March 31, 2023 19:01
@jmeridth
Copy link
Member Author

archived workflows api doesn't allow for additional fields. This is gonna be interesting.

@jmeridth
Copy link
Member Author

jmeridth commented May 8, 2023

This will be affected by #10781

@terrytangyuan
Copy link
Member

This will be impacted by #11121

@jmeridth
Copy link
Member Author

@terrytangyuan 👍 yeah. I'm very excited about your changes. They will give the user a better experience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Display title and description annotations on workflow archive view
3 participants