From cdc2ac2d9b93c8f9765dc8c8a9ded8b378b9bbad Mon Sep 17 00:00:00 2001 From: badrAZ Date: Fri, 6 Jul 2018 11:20:27 +0200 Subject: [PATCH] feat(xo-web/VM/Snapshots): notify the result of reverting a VM (#3150) Fixes #3095 --- CHANGELOG.md | 1 + packages/xo-web/src/common/intl/messages.js | 2 ++ packages/xo-web/src/common/xo/index.js | 7 ++++++- 3 files changed, 9 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index ca05ef056..ac7cf2481 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -10,6 +10,7 @@ - [Remotes] Allow optional port for NFS remote [2299](https://github.com/vatesfr/xen-orchestra/issues/2299) (PR [#3131](https://github.com/vatesfr/xen-orchestra/pull/3131)) - [Backup NG form] Add offline snapshot info (PR [#3144](https://github.com/vatesfr/xen-orchestra/pull/3144)) - [Backup NG overview] Display concurrency and offline snapshot value [3087](https://github.com/vatesfr/xen-orchestra/issues/3087) (PR [3145](https://github.com/vatesfr/xen-orchestra/pull/3145)) +- [VM revert] notify the result of reverting a VM [3095](https://github.com/vatesfr/xen-orchestra/issues/3095) (PR [3150](https://github.com/vatesfr/xen-orchestra/pull/3150)) ### Bug fixes diff --git a/packages/xo-web/src/common/intl/messages.js b/packages/xo-web/src/common/intl/messages.js index 3a52b5a28..a2809f66f 100644 --- a/packages/xo-web/src/common/intl/messages.js +++ b/packages/xo-web/src/common/intl/messages.js @@ -961,6 +961,8 @@ const messages = { snapshotDescription: 'Description', snapshotAction: 'Action', snapshotQuiesce: 'Quiesced snapshot', + vmRevertSuccessfulTitle: 'Revert successful', + vmRevertSuccessfulMessage: 'VM successfully reverted', // ----- VM log tab ----- logRemoveAll: 'Remove all logs', diff --git a/packages/xo-web/src/common/xo/index.js b/packages/xo-web/src/common/xo/index.js index 1e1ce2fdc..710e2f941 100644 --- a/packages/xo-web/src/common/xo/index.js +++ b/packages/xo-web/src/common/xo/index.js @@ -1210,7 +1210,12 @@ export const revertSnapshot = snapshot => body: , }).then( snapshotBefore => - _call('vm.revert', { snapshot: resolveId(snapshot), snapshotBefore }), + _call('vm.revert', { + snapshotBefore, + snapshot: resolveId(snapshot), + }).then(() => + success(_('vmRevertSuccessfulTitle'), _('vmRevertSuccessfulMessage')) + ), noop )