[pve-devel] [PATCH widget-toolkit] fix #5074: notify: sendmail: smtp: fix mailto/mailto-user parameter deletion

Lukas Wagner l.wagner at proxmox.com
Thu Jan 4 14:03:06 CET 2024



On 1/4/24 13:38, Fiona Ebner wrote:
> by doing a cbind of isCreate to the top-level widget so that cbind in
> the nested widgets for deleteEmpty works.
> 
> In the GUI, when a sendmail/smtp target is edited and either
> 'Additional Recipients' or 'Recipients' is completely removed (only
> possible if the other field contains a value), parameter deletion did
> not work properly. After applying the changes, the old value would
> still be in place.
> 

I can confirm that this patch works as expected and fixes the issue.

Tested-by: Lukas Wagner <l.wagner at proxmox.com>
Reviewed-by: Lukas Wagner <l.wagner at proxmox.com>


-- 
- Lukas




More information about the pve-devel mailing list