Re: applied: [PATCH datacenter-manager/proxmox v4 0/7] fix #6914: add option to remove already existing token

[Dominik Csapak <d.csapak@proxmox.com>]

it seems the 2/6 commit is missing here, meaning the ui now uses an
option on delete that is not there in the api, was this
intentional? (i.e. is there something wrong with the patch from
your side?)

On 3/8/26 7:23 PM, Thomas Lamprecht wrote:
> On Wed, 18 Feb 2026 17:41:28 +0100, Shan Shaji wrote:
>> If a user removed a remote without deleting its associated token,
>> PDM would not allow re-adding the same remote unless the token
>> was changed. To fix this, support for optionally deleting the token
>> on the remote has been added.
>>
>> new changes:
>> - use ConfirmDialog widget for creating the confirmation dialog.
>> - add token deletion as opt-out feature.
>>
>> [...]
> 
> Applied, thanks!
> 
> [1/6] server: pbs-client: add method to delete token from remote
>        commit: cef803fde6069c32cb79182336a7f8c62f31342b
> [1/1] pve-api-types: generate missing `delete_token` method
>        (no commit info)
> [3/6] pdm-client: accept `delete-token` argument for deleting api token
>        commit: fa87d7c7b40e89cb528c78ecac217cd31d713061
> [4/6] fix #6914: cli-client: add option to delete token from remote
>        commit: 8249b9a40b0b47f600edc8f552fedd19ac12b4cf
> [5/6] fix #6914: ui: add remove remote dialog with optional token deletion
>        commit: 4ecebbcbdc6c232c8dcc5d45e83bd39d81480991
> [6/6] fix #6914: cli-admin: add option to delete token from remote
>        commit: 461f3331545c4f5c4d21a05f64c8f374fa06ad5e
> 
> 
> 
>