Rename policy property to Update_CheckForUpdatesAtStartup and allow enabling/disabling

Co-authored-by: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>

Author: Copilot

Documentation/config/README.md

@@ -11,21 +11,32 @@ This directory contains an example `config.json` file that can be placed in the
 
 ## Configuration Options
 
-### Update_DisableUpdateCheck
+### Update_CheckForUpdatesAtStartup
 
-When set to `true`, disables the automatic update check at startup for all users.
+Controls whether the application checks for updates at startup for all users. This overrides the user's personal setting.
 
-**Example:**
+- Set to `true` to enable automatic update checks at startup
+- Set to `false` to disable automatic update checks at startup
+- Omit the property to allow users to control this setting themselves
+
+**Example (disable updates):**
+```json
+{
+  "Update_CheckForUpdatesAtStartup": false
+}
+```
+
+**Example (enable updates):**
 ```json
 {
-  "Update_DisableUpdateCheck": true
+  "Update_CheckForUpdatesAtStartup": true
 }
 ```
 
 This is useful for enterprise deployments where you want to:
-- Control software updates centrally
-- Prevent users from being prompted about updates
-- Disable update checks on multiple machines without user intervention
+- Control software update checking behavior centrally
+- Ensure consistent update check behavior across all users
+- Either enforce update checks or prevent them system-wide
 
 ## File Location
 
@@ -36,6 +47,7 @@ The `config.json` file should be placed in:
 ## Notes
 
 - System-wide configuration takes precedence over user settings
+- Users will see a message in the UI indicating when a setting is managed by the administrator
 - If the file doesn't exist or contains invalid JSON, it will be ignored and default user settings will apply
 - Changes to `config.json` require restarting the application to take effect
 - The file is optional - if not present, user settings will be used as normal

Documentation/config/config.json.example

@@ -1,3 +1,3 @@
 {
-  "Update_DisableUpdateCheck": true
+  "Update_CheckForUpdatesAtStartup": false
 }

Source/NETworkManager.Settings/PolicyInfo.cs

@@ -9,9 +9,10 @@ namespace NETworkManager.Settings;
 public class PolicyInfo
 {
     /// <summary>
-    ///     Disable update check for all users. When set to true, the application will not check for updates at startup.
-    ///     This overrides the user's "Update_CheckForUpdatesAtStartup" setting.
+    ///     Controls the "Check for updates at startup" setting for all users.
+    ///     When set, this value overrides the user's "Update_CheckForUpdatesAtStartup" setting.
+    ///     Set to true to enable update checks, false to disable them.
     /// </summary>
-    [JsonPropertyName("Update_DisableUpdateCheck")]
-    public bool? Update_DisableUpdateCheck { get; set; }
+    [JsonPropertyName("Update_CheckForUpdatesAtStartup")]
+    public bool? Update_CheckForUpdatesAtStartup { get; set; }
 }

Source/NETworkManager.Settings/PolicyManager.cs

@@ -73,9 +73,9 @@ public static void Load()
                 Log.Info("System-wide policies loaded successfully.");
 
                 // Log enabled settings
-                if (Current.Update_DisableUpdateCheck.HasValue)
+                if (Current.Update_CheckForUpdatesAtStartup.HasValue)
                 {
-                    Log.Info($"System-wide policy - Update_DisableUpdateCheck: {Current.Update_DisableUpdateCheck.Value}");
+                    Log.Info($"System-wide policy - Update_CheckForUpdatesAtStartup: {Current.Update_CheckForUpdatesAtStartup.Value}");
                 }
             }
             catch (Exception ex)

Source/NETworkManager.Settings/SettingsManager.cs

@@ -64,10 +64,10 @@ public static bool ShouldCheckForUpdatesAtStartup
     {
         get
         {
-            // System-wide policy takes precedence - if it explicitly disables updates, honor it
-            if (PolicyManager.Current?.Update_DisableUpdateCheck == true)
+            // System-wide policy takes precedence - if set, use the policy value
+            if (PolicyManager.Current?.Update_CheckForUpdatesAtStartup.HasValue == true)
             {
-                return false;
+                return PolicyManager.Current.Update_CheckForUpdatesAtStartup.Value;
             }
 
             // Otherwise, use the user's setting

Source/NETworkManager.Settings/config.json.example

@@ -1,3 +1,3 @@
 {
-  "Update_DisableUpdateCheck": true
+  "Update_CheckForUpdatesAtStartup": false
 }

Source/NETworkManager/ViewModels/SettingsUpdateViewModel.cs

@@ -29,7 +29,7 @@ public bool CheckForUpdatesAtStartup
     /// <summary>
     ///     Gets whether the "Check for updates at startup" setting is managed by system-wide policy.
     /// </summary>
-    public bool IsUpdateCheckManagedByPolicy => PolicyManager.Current?.Update_DisableUpdateCheck.HasValue == true;
+    public bool IsUpdateCheckManagedByPolicy => PolicyManager.Current?.Update_CheckForUpdatesAtStartup.HasValue == true;
 
     private bool _checkForPreReleases;
 
@@ -83,7 +83,9 @@ public SettingsUpdateViewModel()
 
     private void LoadSettings()
     {
-        CheckForUpdatesAtStartup = SettingsManager.Current.Update_CheckForUpdatesAtStartup;
+        // If policy is set, show the policy value; otherwise show the user's setting
+        CheckForUpdatesAtStartup = PolicyManager.Current?.Update_CheckForUpdatesAtStartup 
+                                    ?? SettingsManager.Current.Update_CheckForUpdatesAtStartup;
         CheckForPreReleases = SettingsManager.Current.Update_CheckForPreReleases;
         EnableExperimentalFeatures = SettingsManager.Current.Experimental_EnableExperimentalFeatures;
     }