fix(ui): merge WF List FTU Panel with New Version Modal #11742
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #10821
Technically a follow-up to #8596, which added this FTU Panel
Motivation
Modifications
Move the description of the new default instead to the New Version Modal
unrelated: simplify some syntax in this file using optional chaining (from investigating UI does not show all running workflows (it used to) / UI and
argo list
differ (missing running workflows) #9696)Verification
/workflows
See below screenshot with the New Version Modal as well as the Workflow List in the background:
Future Work
WorkflowList
could be optimized a bit. It's got a few internal functions that are purely static and could be moved out of the component as such. Left the diff pretty minimal in this PR thoughEDIT: Completed this in #11891