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

Drop VM detail edit values on cancel #396

Merged
merged 1 commit into from
Apr 24, 2019

Conversation

pcbailey
Copy link
Contributor

If the user entered values into fields in the edit mode of the VM details screen, clicked the Cancel button, and then re-entered edit mode, the previously entered values would still be populated. This PR backs up the saved values before entering edit mode and restores them if editing is cancelled.

Bug: https://bugzilla.redhat.com/1698474

@coveralls
Copy link

coveralls commented Apr 22, 2019

Pull Request Test Coverage Report for Build 1412

  • 5 of 5 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.005%) to 86.469%

Totals Coverage Status
Change from base Build 1408: 0.005%
Covered Lines: 3645
Relevant Lines: 4040

💛 - Coveralls

src/components/Details/VmDetails/VmDetails.js Outdated Show resolved Hide resolved
src/components/Details/VmDetails/VmDetails.js Outdated Show resolved Hide resolved
@pcbailey pcbailey force-pushed the clear-flavor-changes-on-cancel branch from bf60ce6 to 521bcca Compare April 23, 2019 13:32
@pcbailey pcbailey requested a review from atiratree April 23, 2019 13:34
@pcbailey pcbailey force-pushed the clear-flavor-changes-on-cancel branch from 521bcca to 4efb873 Compare April 24, 2019 13:24
@atiratree atiratree merged commit dd58638 into kubevirt:master Apr 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants