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
Bug 1830929: Feat(pl-filter-status): Add others as a filter status #5245
Bug 1830929: Feat(pl-filter-status): Add others as a filter status #5245
Conversation
42869d5
to
ef9f416
Compare
ef9f416
to
20a6a9f
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.
@abhinandan13jan Could you also handle Pipeline Runs? It has a similar look and feel and should be upgraded at the same time. I believe it's the same content as the Pipeline Details => Pipeline Runs tab as well, but please apply there as well if not.
Others include only the Pipelines, which have no runs associated with it... i.e no status. Should we cascade the others status as a default/safety block only? As Pipeline runs will have a status of their own and will not fall in the 'others' category.. @andrewballantyne |
/retitle Bug 1830929: Feat(pl-filter-status): Add others as a filter status |
@abhinandan13jan: This pull request references Bugzilla bug 1830929, which is valid. The bug has been moved to the POST state. 3 validation(s) were run on this bug
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. |
/test analyze |
Fair @abhinandan13jan - my mistake. I was thinking of the "catch all" net that |
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
@openshift/team-devconsole-ux FYI I don't think we need to hold up the PR as they did approve the solution on the ticket. But still worth tagging them on PRs @abhinandan13jan |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abhinandan13jan, andrewballantyne 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 |
@abhinandan13jan: All pull requests linked via external trackers have merged: openshift/console#5245. Bugzilla bug 1830929 has been moved to the MODIFIED state. 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. |
Fixes:
https://issues.redhat.com/browse/ODC-2401
Analysis / Root cause:
Other status to be set as a Pipeline List filter
Solution Description:
Added a new filter status Other to filters of Pipeline List
Screenshot
Browser conformation
Chrome 73