rhtyd commented on a change in pull request #289: view: fix list group action
URL: https://github.com/apache/cloudstack-primate/pull/289#discussion_r408302100
##########
File path: src/views/AutogenView.vue
##########
@@ -726,6 +741,11 @@ export default {
var hasJobId = false
api(this.currentAction.api, params).then(json => {
// set action data for reload tree-view
+ if (this.currentAction.api.startsWith('delete') ||
Review comment:
I would prefer that we simply run the api of the action against all the
selected items; the only exception could be if the parameter the API accepts is
`ids` then instead of looping+calling on all items; we can simply call a single
API. Let's discuss.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
