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
adds decorator if any associated Workload pods has type warning #4052
adds decorator if any associated Workload pods has type warning #4052
Conversation
df77dd0
to
1f96b3a
Compare
/kind feature |
frontend/packages/dev-console/src/components/topology/components/nodes/WorkloadNode.tsx
Outdated
Show resolved
Hide resolved
1f96b3a
to
6825772
Compare
@invincibleJai for warning levels, please use the warning-triangle icon in orange (CSS var is --pf-c-alert--m-inline--m-warning__icon--Color). Later when we have critical alerts, we would offer the ! in red thanks! |
6825772
to
b593e49
Compare
@serenamarie125 thanks, have made the changes and updated gif as well PTAL. Screenshot: |
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.
Perfect! Thanks for making the change, this looks great!
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
element | ||
.getGraph() | ||
.getController() | ||
.fireEvent('selection', [element.getId()]); |
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.
Interesting approach. It'll work :)
Use the const for the selection event instead of a string
x: number; | ||
y: number; | ||
element: Node; | ||
onClickTab?: (name: string) => void; |
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.
Rename to just onClick
with no name param. The name is a const, use it in the connect down below.
}; | ||
|
||
const mapDispatchToProps = (dispatch) => ({ | ||
onClickTab: (name) => dispatch(UIActions.selectOverviewDetailsTab(name)), |
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.
onClick: () => dispatch(UIActions.selectOverviewDetailsTab('Monitoring)),
b593e49
to
62cc2f1
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
62cc2f1
to
8ce4b13
Compare
/approve |
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
/retest Please review the full test history for this PR and help us cut down flakes. |
8ce4b13
to
8de790a
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
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abhi-kn, christianvogt, invincibleJai, serenamarie125, 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 |
adds decorator if any associated Workload pods have type warning and on click open sidebar
Tracks: https://issues.redhat.com/browse/ODC-2587
Gif :
cc: @openshift/team-devconsole-ux @serenamarie125