Skip to content
This repository has been archived by the owner on Jan 12, 2023. It is now read-only.

Bug 1953253: feat(vms): add an icon indicator to reflect vms power status #801

Merged
merged 5 commits into from Oct 12, 2021

Conversation

seanforyou23
Copy link
Contributor

This PR adds an icon to the VM name column that reflects its power status. With a nested tooltip like this, we no longer get the tippifying of the vm name for free, so I had to double wrap both the icon and the vm name separately to ensure they would both display.

Screen Shot 2021-10-11 at 4 15 14 PM

Should help close #594

@seanforyou23 seanforyou23 requested review from mturley, gildub, vconzola and a team October 11, 2021 20:28
@github-actions
Copy link

Valid bug 1953253

@konveyor-preview-bot
Copy link

🚀 Deployed Preview: http://konveyor-forklift-ui-pr-801-preview.surge.sh

Compare with current main branch: http://konveyor-forklift-ui-preview.surge.sh

Copy link
Collaborator

@mturley mturley left a comment

Choose a reason for hiding this comment

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

Looking good, some small things.

src/app/Plans/components/Wizard/SelectVMsForm.tsx Outdated Show resolved Hide resolved
src/app/Plans/components/Wizard/SelectVMsForm.tsx Outdated Show resolved Hide resolved
src/app/queries/types/vms.types.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@gildub gildub left a comment

Choose a reason for hiding this comment

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

Looks good. The only thing is about the mock data, I wonder why we're having only PoweredOn VMs, could we mix up some VMs being off?

Copy link
Collaborator

@mturley mturley left a comment

Choose a reason for hiding this comment

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

One more thing, don't kill me...

src/app/common/components/VMNameWithPowerState.tsx Outdated Show resolved Hide resolved
Copy link
Collaborator

@mturley mturley left a comment

Choose a reason for hiding this comment

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

LGTM!

@sonarcloud
Copy link

sonarcloud bot commented Oct 12, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 5 Code Smells

No Coverage information No Coverage information
2.2% 2.2% Duplication

@mturley mturley merged commit 1639e60 into kubev2v:main Oct 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BZ#1953253] Show VM power status while selecting VMs during plan creation
4 participants