[pve-devel] applied: [PATCH manager v4 0/2] fix #474: allow transfer from container/vms

[Wolfgang Bumiller]

applied, thanks

@Dominik: does extjs have an 'enableFn' for rows in a grid?
IMO we should either disable the ones with pools when the transfer
checkbox is not checked, or hide them (but when hiding them after
already checking them... it's weird)
Or disable the 'Add' button if a VM with a pool is checked?

On Thu, Aug 10, 2023 at 12:09:00PM +0200, Philipp Hufnagl wrote:
> When a member of 2 pools wants to transfer a
>  vm/container to an other pool they can not do that. The vm/container would
>  have first to be removed form the current pool resulting in a loss of
>  privileges of the pool member for this vm/contianer. This feature introduces
>  a way to transfer a vm between pools, guarded by a checkbox from accidental
>  transfers
> 
> changes to v4:
> * check for allocate permissions of the originating pool
> 
> changes to v3:
> * fix subject typo
> * at version log
> 
> changes to v2:
> * split patch in front and backend
> 
> Philipp Hufnagl (2):
>   fix #474: api: allow transfer from container/vms
>   fix #474: ui: allow transfer from container/vms
> 
>  PVE/API2/Pool.pm                 | 16 ++++++++++++++--
>  www/manager6/grid/PoolMembers.js | 17 ++++++++++++++---
>  2 files changed, 28 insertions(+), 5 deletions(-)
> 
> -- 
> 2.39.2
> 
> 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 
>