Move initial setup of IsAnalyticsReportingApproved to PreferenceSetti…

…ngs constructor (#9666)
DynamoDS · May 29, 2019 · e475807 · e475807
1 parent 351c88b
commit e475807
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/src/DynamoCore/Configuration/PreferenceSettings.cs b/src/DynamoCore/Configuration/PreferenceSettings.cs
@@ -322,6 +322,7 @@ public PreferenceSettings()
             // Default Settings
             IsFirstRun = true;
             IsUsageReportingApproved = false;
+            IsAnalyticsReportingApproved = true;
             LibraryWidth = 304;
             ConsoleHeight = 0;
             ShowPreviewBubbles = true;

diff --git a/src/DynamoCoreWpf/Services/UsageReportingManager.cs b/src/DynamoCoreWpf/Services/UsageReportingManager.cs
@@ -152,9 +152,6 @@ public void CheckIsFirstRun(Window ownerWindow, IBrandingResourceProvider resour
             {
                 FirstRun = false;
 
-                //Analytics enable by defaultwa
-                IsAnalyticsReportingApproved = true;
-
                 //Prompt user for detailed reporting
                 if (!DynamoModel.IsTestMode)
                     ShowUsageReportingPrompt(ownerWindow);