Skip to content
This repository was archived by the owner on Apr 28, 2020. It is now read-only.

[WIP] Added top consumers card #334

Closed
wants to merge 1 commit into from

Conversation

afreen23
Copy link
Contributor

@afreen23 afreen23 commented Apr 9, 2019

No description provided.

@coveralls
Copy link

coveralls commented Apr 9, 2019

Pull Request Test Coverage Report for Build 1379

  • 19 of 21 (90.48%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.01%) to 87.322%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/components/StorageOverview/TopConsumers/TopConsumers.js 9 11 81.82%
Totals Coverage Status
Change from base Build 1366: 0.01%
Covered Lines: 3273
Relevant Lines: 3593

💛 - Coveralls


import { getTopConsumerVectorStats } from '../../../selectors';

const TopConsumersBody = ({ columns }) => (
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems like a different component than Dashboard/TopConsumers. Can you rename it a bit? and maybe add it as a generic component to Dashboard?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok..i have just rebased

@kubevirt-bot kubevirt-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 10, 2019
@afreen23 afreen23 force-pushed the top-consumers-card branch from ef35f69 to f4b74c5 Compare April 11, 2019 07:22
@kubevirt-bot kubevirt-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Apr 11, 2019
@gnehapk
Copy link
Contributor

gnehapk commented Apr 12, 2019

Closing this PR as its changes are superseded by #349

@afreen23 afreen23 closed this Apr 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants