From fdb0453a42a9a697ae62ead1b59065da9f6dec9a Mon Sep 17 00:00:00 2001 From: Pulse Monitor Date: Fri, 29 Aug 2025 20:40:57 +0000 Subject: [PATCH] fix: actually prompt for auto-updates in container installation - Add auto-updates prompt to Quick mode (not just Advanced) - Remove 'local' keyword so the flag variable is accessible later - Previously the prompt appeared but the choice wasn't being passed through --- install.sh | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/install.sh b/install.sh index eb11753e4..00f19dffe 100755 --- a/install.sh +++ b/install.sh @@ -228,7 +228,7 @@ create_lxc_container() { echo "Enable automatic updates?" echo "Pulse can automatically install stable updates daily (between 2-6 AM)" safe_read_with_default "Enable auto-updates? [y/N]: " enable_updates "n" - local auto_updates_flag="" + auto_updates_flag="" if [[ "$enable_updates" =~ ^[Yy]$ ]]; then auto_updates_flag="--enable-auto-updates" fi @@ -329,7 +329,15 @@ create_lxc_container() { firewall=1 unprivileged=1 frontend_port=7655 - auto_updates_flag="" # Quick mode defaults to no auto-updates + # Quick mode should ask about auto-updates too + echo + echo "Enable automatic updates?" + echo "Pulse can automatically install stable updates daily (between 2-6 AM)" + safe_read_with_default "Enable auto-updates? [y/N]: " enable_updates "n" + auto_updates_flag="" + if [[ "$enable_updates" =~ ^[Yy]$ ]]; then + auto_updates_flag="--enable-auto-updates" + fi fi # Get available network bridges