On Mon Jan 19, 2026 at 12:48 PM CET, Fiona Ebner wrote:
> Nit: I feel like the $delete belongs inside the $changes for a clean
> interface. So something like
> $changes = {
> $sid => {
> param => $param,
> delete => $delete,
> }
> }
>
> I know the single caller using bulk won't need it, but the current
> interface does feel slightly off to me. What do you think?
Sounds good, will do! Just for clarification: Is `param => $param` a
single changed property or is $param the hash of changes?
On Mon Jan 19, 2026 at 12:48 PM CET, Fiona Ebner wrote:
>
> Nit: I'd prefer to iterate after 'sort'
Good idea, will do for both!
_______________________________________________
pve-devel mailing list
[email protected]
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
