fix: updateCfg merge default

IvanJosipovic · Dec 18, 2023 · 4889b9e · 4889b9e
1 parent f7814ad
commit 4889b9e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/BlazorApplicationInsights/ApplicationInsights.cs b/src/BlazorApplicationInsights/ApplicationInsights.cs
@@ -78,7 +78,7 @@ public async Task StopTrackEvent(string name, Dictionary<string, object?>? prope
         => await _jsRuntime.InvokeVoidAsync("appInsights.stopTrackEvent", name, properties, measurements);
 
     /// <inheritdoc />
-    public async Task UpdateCfg(Config newConfig, bool? mergeExisting = null)
+    public async Task UpdateCfg(Config newConfig, bool mergeExisting = true)
         => await _jsRuntime.InvokeVoidAsync("appInsights.updateCfg", newConfig, mergeExisting);
 
     /// <inheritdoc />

diff --git a/src/BlazorApplicationInsights/Interfaces/IApplicationInsights.cs b/src/BlazorApplicationInsights/Interfaces/IApplicationInsights.cs
@@ -36,7 +36,7 @@ public interface IApplicationInsights : IAppInsights, IDependenciesPlugin, IProp
     /// </summary>
     /// <param name="newConfig">The new configuration is apply</param>
     /// <param name="mergeExisting">Should the new configuration merge with the existing or just replace it. Default is to merge.</param>
-    Task UpdateCfg(Config newConfig, bool? mergeExisting = null);
+    Task UpdateCfg(Config newConfig, bool mergeExisting = true);
 
     /// <summary>
     /// Sets JSRuntime for the service