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
Create multi tab list component and using that render various pipeline lists #5700
Create multi tab list component and using that render various pipeline lists #5700
Conversation
1ff3d31
to
8a57d38
Compare
ee8d9af
to
5ff3e1b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the component, just a few changes and some other comments. Definitely coming together as I imagined it 👍
frontend/packages/dev-console/src/components/multi-tab-list/MultiTabListPage.scss
Outdated
Show resolved
Hide resolved
frontend/packages/dev-console/src/components/multi-tab-list/MultiTabListPage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/dev-console/src/components/multi-tab-list/MultiTabListPage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/dev-console/src/components/multi-tab-list/MultiTabListPage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/dev-console/src/components/multi-tab-list/MultiTabListPage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/dev-console/src/components/pipelines-lists/PipelinesListsPage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/dev-console/src/components/pipelines-lists/PipelinesListsPage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/dev-console/src/components/pipelineruns/PipelineRunsResourceList.tsx
Outdated
Show resolved
Hide resolved
...ckages/dev-console/src/components/pipeline-resources/list-page/PipelineResourcesListPage.tsx
Show resolved
Hide resolved
b3524e7
to
a2f30a5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple final changes/questions to the way MultiTabListPage
works but otherwise this looks good.
frontend/packages/dev-console/src/components/multi-tab-list/MultiTabListPage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/dev-console/src/components/multi-tab-list/MultiTabListPage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/dev-console/src/components/multi-tab-list/MultiTabListPage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/dev-console/src/components/pipelineruns/PipelineRunsResourceList.tsx
Outdated
Show resolved
Hide resolved
c98510e
to
bdba469
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/assign @christianvogt Because of the logic change in the list page, we'll need a review from Christian. |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andrewballantyne, christianvogt, divyanshiGupta The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest Please review the full test history for this PR and help us cut down flakes. |
2 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
Story: https://issues.redhat.com/browse/ODC-4006
Additional Sub-tasks: https://issues.redhat.com/browse/ODC-4142, https://issues.redhat.com/browse/ODC-4143
This PR adds a new multi tab list page which is being used to render various pipeline lists. This PR also updates pipelines nav.
Note-
Will be adding PipelineResources filters in the upcoming PR for story ODC-4007 which is about adding the pages.
Screenshot: