tiago/pve-manager
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

ui: remove handling of obsolete notification-policy/target settings

Browse source 
These were only used in the 'old' revamped notification stack which was
briefly available on pvetest. With PVE 9 we can finally get completely
rid of these.

Signed-off-by: Lukas Wagner <l.wagner@proxmox.com>
Reviewed-by: Michael Köppl <m.koeppl@proxmox.com>
Tested-by: Michael Köppl <m.koeppl@proxmox.com>
Link: https://lore.proxmox.com/20250709081432.91868-2-l.wagner@proxmox.com
This commit is contained in:
Lukas Wagner 2025-07-09 10:14:31 +02:00 • committed by Thomas Lamprecht
parent 68cd0f25db
commit df147ed0ff
1 changed files with 0 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

16
www/manager6/dc/Backup.js
View file

@ -37,14 +37,6 @@ Ext.define('PVE.dc.BackupEdit', {
delete values.node;
}
// Get rid of new-old parameters for notification settings.
// These should only be set for those selected few who ran
// pve-manager from pvetest.
if (!isCreate) {
Proxmox.Utils.assemble_field_data(values, { delete: 'notification-policy' });
Proxmox.Utils.assemble_field_data(values, { delete: 'notification-target' });
}
let selMode = values.selMode;
delete values.selMode;
@ -158,14 +150,6 @@ Ext.define('PVE.dc.BackupEdit', {
let me = this;
let viewModel = me.getViewModel();
// Migrate 'new'-old notification-policy back to old-old mailnotification.
// Only should affect users who used pve-manager from pvetest. This was a remnant of
// notifications before the overhaul.
let policy = data['notification-policy'];
if (policy === 'always' || policy === 'failure') {
data.mailnotification = policy;
}
if (data.exclude) {
data.vmid = data.exclude;
data.selMode = 'exclude';