"Delete VIP" is not a batch action so it makes sense to use
tables.Action instead of tables.DeleteAction to prevent extra
success message which is in fact incorrect. "Delete VIP" is
a specific action: there is no separate tab for VIPs. That's
why pool's id is used for this message. Moreover, a default
confirmation message is quite confusing too: "You have selected
<pool name associated with VIP>". This patch adds a custom help
text with VIP id to this message.
Reviewed: https:/ /review. openstack. org/184776 /git.openstack. org/cgit/ openstack/ horizon/ commit/ ?id=05a5ae38774 b78ee8ed76b3d34 7c1ca89dd2fa07
Committed: https:/
Submitter: Jenkins
Branch: master
commit 05a5ae38774b78e e8ed76b3d347c1c a89dd2fa07
Author: Tatiana Ovchinnikova <email address hidden>
Date: Wed Aug 12 11:46:26 2015 +0300
Delete VIP action refactoring
"Delete VIP" is not a batch action so it makes sense to use
tables.Action instead of tables.DeleteAction to prevent extra
success message which is in fact incorrect. "Delete VIP" is
a specific action: there is no separate tab for VIPs. That's
why pool's id is used for this message. Moreover, a default
confirmation message is quite confusing too: "You have selected
<pool name associated with VIP>". This patch adds a custom help
text with VIP id to this message.
Closes-Bug: #1439180
Change-Id: I74c13bcbee56e2 2d8814e8a4118c9 06c23e2405e