[pdm-devel] [PATCH proxmox{-yew-comp, -datacenter-manager} v2 0/7] PVE node update view

[Thomas Lamprecht]

Am 04.09.25 um 11:30 schrieb Dominik Csapak:
> in general the code and ui LGTM (with some minor cleanup potential)
> 
> the only thing irking me is that the upgrade button is disabled
> without further explanation for the user.
> 
> @Thomas
> i would apply this, and send some follow ups,
> especially one where we overwrite the upgrade button
> to open the nodes upgrade panel on the pve ui in a new tab for now
> until we can tunnel the websocket and show it locally, what do you say?

That's exactly what I talked with Lukas yesterday and indeed an OK
stop gap. FWIW, we also briefly discussed the option of directly
opening the target remotes xterm.js upgrade console, potentially
providing both options (e.g. as split button).

> alternatively, we can hide the button or the whole taskbar, but i would
> not simply disable the upgrade button without any explanation or
> way to do it...
> 

No, some way to navigate to the target remote would be definitively
good to have.