[GitHub] [cloudstack] utchoang commented on a change in pull request #6066: UI: Fix issue on volume snapshots wizard

[GitBox <gi...@apache.org>]

utchoang commented on a change in pull request #6066:
URL: https://github.com/apache/cloudstack/pull/6066#discussion_r821276309



##########
File path: ui/src/views/storage/TakeSnapshot.vue
##########
@@ -145,7 +153,7 @@ export default {
         }
 
         let params = {}
-        params.volumeId = this.resource.id
+        params.volumeId = this.resource.id ? this.resource.id : this.oldResource.id

Review comment:
       This problem comes from the pollJob completed and cleared the resource. In my opinion, this case will appear a lot with other actions, not just Take Snapshot. My suggestion to solve it is to add the condition to `Autogenview.vue` line 924 with the following condition: `if (!this.showAction || this.dataView)`.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org