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
ODC-7281: Provide Column management option for the TaskRuns list page #12689
ODC-7281: Provide Column management option for the TaskRuns list page #12689
Conversation
@aneeshmbhat: This pull request references ODC-7281 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/retest |
@aneeshmbhat Frontend tests are failing because the Translation file is not up to date. Please Run |
903bbcb
to
928966a
Compare
frontend/packages/pipelines-plugin/src/components/taskruns/list-page/TaskRunsListPage.tsx
Outdated
Show resolved
Hide resolved
...ges/pipelines-plugin/integration-tests/features/pipelines/pipeline-task-runs-display.feature
Outdated
Show resolved
Hide resolved
frontend/packages/pipelines-plugin/src/components/taskruns/list-page/TaskRunsListPage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/pipelines-plugin/src/components/taskruns/list-page/TaskRunsListPage.tsx
Outdated
Show resolved
Hide resolved
frontend/packages/pipelines-plugin/src/components/taskruns/list-page/TaskRunsList.tsx
Outdated
Show resolved
Hide resolved
7bb44c4
to
354ea5b
Compare
frontend/packages/pipelines-plugin/src/components/taskruns/list-page/TaskRunsListPage.tsx
Outdated
Show resolved
Hide resolved
354ea5b
to
9a73fbd
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.
Propagating docs and px acks from the Epic |
And user clicks on TaskRuns tab | ||
And user clicks manage columns button | ||
And user enables Duration column |
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.
Duration column has been already enabled here, so no need to check duration option again, instead we can verify that duration option is already checked.
|
||
@regression | ||
Scenario: Check for Duration column when enabled |
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.
Add test cases number accordingly. (P-05-TC07)
9a73fbd
to
34e6ab2
Compare
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aneeshmbhat, hemantsaini-7, vikram-raj 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 |
@aneeshmbhat: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Story:
https://issues.redhat.com/browse/ODC-7281
Description:
Added Button with associated Modal for handling Column Management
Screen shots / Gifs for design review:
ODC-7281.mp4
Unit test coverage report:
WIP
Test setup:
Browser conformance: