You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by ga...@apache.org on 2019/01/09 10:28:54 UTC

[cloudstack] branch master updated: Rework for destroy VM with volumes UI (#3120)

This is an automated email from the ASF dual-hosted git repository.

gabriel pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git


The following commit(s) were added to refs/heads/master by this push:
     new 7b0ff7f  Rework for destroy VM with volumes UI (#3120)
7b0ff7f is described below

commit 7b0ff7f0c7e0a9643f1e1f11cfe6a2c652ccfe16
Author: Dingane Hlaluku <di...@shapeblue.com>
AuthorDate: Wed Jan 9 12:28:47 2019 +0200

    Rework for destroy VM with volumes UI (#3120)
    
    * Remove redundant checkbox that controls the visibility of volumes to delete when destroying VMs
    
    * Change label from Volume IDs to Delete Volumes
    
    * Revert back deleted Volume IDs label
---
 ui/scripts/instances.js | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/ui/scripts/instances.js b/ui/scripts/instances.js
index 43e6bc9..7a959df 100644
--- a/ui/scripts/instances.js
+++ b/ui/scripts/instances.js
@@ -116,13 +116,14 @@
                     volumes: {
                         label: 'label.delete.volumes',
                         isBoolean: true,
-                        isChecked: false
+                        isChecked: true,
+                        isHidden: true,
                     },
                     volumeids: {
-                        label: 'label.volume.ids',
+                        label: 'label.delete.volumes',
                         dependsOn: 'volumes',
                         isBoolean: true,
-                        isHidden: true,
+                        isHidden: false,
                         emptyMessage: 'label.volume.empty',
                         multiDataArray: true,
                         multiData: function(args) {