Re: [PATCH datacenter-manager 15/16] ui: auto-installer: wizard: add checkbox to add target as new remote

["Christoph Heiss" <c.heiss@proxmox.com>]

Thanks for the review!

On Tue Aug 4, 2026 at 1:39 PM CEST, Lukas Wagner wrote:
> On Fri Jul 31, 2026 at 4:35 PM CEST, Christoph Heiss wrote:
>> Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>
>> ---
>>  .../remotes/auto_installer/prepared_answer_add_wizard.rs   | 1 +
>>  ui/src/remotes/auto_installer/prepared_answer_form.rs      | 7 +++++++
>>  2 files changed, 8 insertions(+)
>>
>> diff --git a/ui/src/remotes/auto_installer/prepared_answer_add_wizard.rs b/ui/src/remotes/auto_installer/prepared_answer_add_wizard.rs
>> index 03d78be8..a4ea66e0 100644
>> --- a/ui/src/remotes/auto_installer/prepared_answer_add_wizard.rs
>> +++ b/ui/src/remotes/auto_installer/prepared_answer_add_wizard.rs
>> @@ -81,6 +81,7 @@ impl AddAnswerWizardProperties {
>>              // post hook
>>              post_hook_base_url: pdm_origin(),
>>              post_hook_cert_fp: fingerprint,
>> +            post_hook_add_as_remote: true,
>>              // templating
>>              template_counters,
>>              // subscription
>> diff --git a/ui/src/remotes/auto_installer/prepared_answer_form.rs b/ui/src/remotes/auto_installer/prepared_answer_form.rs
>> index 6ee4b15a..e6eace8f 100644
>> --- a/ui/src/remotes/auto_installer/prepared_answer_form.rs
>> +++ b/ui/src/remotes/auto_installer/prepared_answer_form.rs
>> @@ -985,6 +985,13 @@ pub fn render_auth_form(
>>                  .tip(tr!("Optional certificate fingerprint"))
>>                  .value(config.post_hook_cert_fp.clone()),
>>          )
>> +        .with_field(
>> +            tr!("Provision As New Remote"),
>> +            Checkbox::new()
>> +                .name("post-hook-add-as-remote")
>> +                .tip(tr!("Only available for Virtual Environment and Backup Server. Add the target host as new remote to this Datacenter Manager instance."))
>
> In other words, available for all products PDM currently supports? I
> think you could leave out the tooltip, this might actually confuse
> users.

Yes. Thought I state it explicitly, since the auto-installer can be used
for PMG and PDM as well.

But I see that this is also kind of implied, that it will only work for
products which are (currently) supported by PDM.

>
>> +                .default(config.post_hook_add_as_remote),
>> +        )
>>          .with_large_custom_child(
>>              Container::from_tag("p")
>>                  .key("post-hook-hint")