[pve-devel] [PATCH widget-toolkit/manager] improve combogrid default value handling
Fabian Grünbichler
f.gruenbichler at proxmox.com
Wed Jul 19 14:22:54 CEST 2023
somewhat plays into #4840 , in that we now no longer do a bogus GET call
for the 'null' storage and display a misleading permission error as a
result, but there still could be a nice (big?) error message about the
general lack of (accessible) storages ;)
https://bugzilla.proxmox.com/show_bug.cgi?id=4840
On July 19, 2023 2:11 pm, Dominik Csapak wrote:
> 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
>
>
>
> _______________________________________________
> pve-devel mailing list
> pve-devel at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
>
>
>
More information about the pve-devel
mailing list