[pdm-devel] applied: [PATCH datacenter-manager 1/1] ui: remote wizard: validate remote ID
Thomas Lamprecht
t.lamprecht at proxmox.com
Wed Oct 29 15:09:28 CET 2025
On Wed, 29 Oct 2025 14:46:27 +0100, Fiona Ebner wrote:
> As reported in the community forum [0], the remote ID field in the
> remote wizard was not validated. When using an inavlid remote ID, only
> the backend would report an error upon submission. Add validation to
> the 'Remote ID' field to catch this early.
>
> [0]: https://forum.proxmox.com/threads/171742/post-808943
>
> [...]
Applied, thanks!
Tiny nit, but as I noticed it more than a few times now already:
Personally I'm not a big fan of cover-letters for simple single-patch
submissions, just adds noise. Basically I'd prefer:
- single commit -> single patch and no cover letter, it's basically just a
`git send-email -1` call anyway.
- multiple commits -> always a cover letter.
But if I'd have to chose between either always cover-letter or never I'd go for
always, so it's fine for me, just find it unnecessary and cost my slightly more
time to read a extra post to see if there is anything that special that
warrants a cover-letter for "just" a single patch.
[1/1] ui: remote wizard: validate remote ID
commit: 0e013085a1d6ed8b70cdb16f4d51af4f6b2f301f
More information about the pdm-devel
mailing list