[pve-devel] [PATCH widget-toolkit/manager] improve combogrid default value handling
Dominik Csapak
d.csapak at proxmox.com
Wed Jul 19 14:11:08 CEST 2023
the default value for combogrids is '', but we often need to set it to
[], to avoid issues with dirty tracking. Fix this by setting it to [] by
default, making it unnecessary to carry the workaround + comment around
in child classes.
the first patch of pve-manager is a bit unrelated but popped up during
development (can be applied independently)
the second patch of pve-manager is also a bit independent, but it fixes
wrong dirty tracking in the firewall rule edit window iff the widget
toolkit patch is also present
the third manager patch depends on the widget toolkit patch
proxmox-widget-toolkit:
Dominik Csapak (1):
combogrid: initialze value with [] by default
src/form/ComboGrid.js | 2 +-
src/form/NetworkSelector.js | 4 ----
2 files changed, 1 insertion(+), 5 deletions(-)
pve-manager:
Dominik Csapak (3):
ui: ipset: make ip/cidr required
ui: don't set the default value of combogrids to ''
ui: don't set the default value of combogrids to []
www/manager6/form/NodeSelector.js | 5 +----
www/manager6/grid/FirewallRules.js | 2 --
www/manager6/ha/GroupSelector.js | 1 -
www/manager6/panel/IPSet.js | 2 +-
4 files changed, 2 insertions(+), 8 deletions(-)
--
2.30.2
More information about the pve-devel
mailing list