-
Notifications
You must be signed in to change notification settings - Fork 30
Conversation
Pull Request Test Coverage Report for Build 1344
💛 - Coveralls |
Pull Request Test Coverage Report for Build 1259
💛 - Coveralls |
fbb7328
to
85b5c58
Compare
85b5c58
to
6ec0cac
Compare
Ref: kubevirt/web-ui#312 |
eb69abe
to
783e0a2
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.
ClusterOverview/Health/HealthBody
is still used in Compliance
card - can you convert it to Dashboard/Health/HealthBody
and them remove ClusterOverview/Health/HealthBody
?
title: PropTypes.string.isRequired, | ||
}; | ||
|
||
export default DashboardCardTitleSeeAll; |
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.
can you remove this default export and export DashboardCardTitleSeeAll itself ? we usually do not use default exports
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.
Sure 👍
@@ -0,0 +1,46 @@ | |||
import React from 'react'; |
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.
SubsystemHealth component should go under Dashboard folder
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.
👍
moved subsystem into dashboard, @rawagner please re review. |
Add subsystem health
Requires kubevirt/web-ui#312
Ref: https://jira.coreos.com/browse/CNV-1151
Screenshots:
