all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [PATCH datacenter-manager] ui: auto-installer: set reboot-on-error value from config
@ 2026-05-28  9:48 Daniel Kral
  2026-05-28 21:37 ` applied: " Thomas Lamprecht
  0 siblings, 1 reply; 2+ messages in thread
From: Daniel Kral @ 2026-05-28  9:48 UTC (permalink / raw)
  To: pdm-devel

Otherwise the "Reboot on Error" checkbox keeps unchecked even though the
value is actually set.

Reported-by: Erik Fastermann <e.fastermann@proxmox.com>
Signed-off-by: Daniel Kral <d.kral@proxmox.com>
---
I had a quick look over the other UI components with highlights on
/(value|default|selected_keys)/ and the other UI components all seem to
have a value set by the config.

 ui/src/remotes/auto_installer/prepared_answer_form.rs | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/ui/src/remotes/auto_installer/prepared_answer_form.rs b/ui/src/remotes/auto_installer/prepared_answer_form.rs
index 560503a..dd0a87e 100644
--- a/ui/src/remotes/auto_installer/prepared_answer_form.rs
+++ b/ui/src/remotes/auto_installer/prepared_answer_form.rs
@@ -282,7 +282,9 @@ pub fn render_global_options_form(
         )
         .with_field(
             tr!("Reboot on Error"),
-            Checkbox::new().name("reboot-on-error"),
+            Checkbox::new()
+                .name("reboot-on-error")
+                .default(config.reboot_on_error),
         )
         .with_field(
             tr!("Post-Installation Action"),

-- 
2.47.3





^ permalink raw reply related	[flat|nested] 2+ messages in thread

* applied: [PATCH datacenter-manager] ui: auto-installer: set reboot-on-error value from config
  2026-05-28  9:48 [PATCH datacenter-manager] ui: auto-installer: set reboot-on-error value from config Daniel Kral
@ 2026-05-28 21:37 ` Thomas Lamprecht
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Lamprecht @ 2026-05-28 21:37 UTC (permalink / raw)
  To: pdm-devel, Daniel Kral

On Thu, 28 May 2026 11:48:26 +0200, Daniel Kral wrote:
> Otherwise the "Reboot on Error" checkbox keeps unchecked even though the
> value is actually set.

Applied, thanks!

[1/1] ui: auto-installer: set reboot-on-error value from config
      commit: 0ddd4ca2cfdc67f960afc120f55d27f1987e6b93




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-05-28 21:37 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-28  9:48 [PATCH datacenter-manager] ui: auto-installer: set reboot-on-error value from config Daniel Kral
2026-05-28 21:37 ` applied: " Thomas Lamprecht

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal