update

Author: reddyashish

src/DynamoCoreWpf/ViewModels/Core/DynamoViewModel.cs

@@ -3333,7 +3333,7 @@ private void CloseHomeWorkspace(object parameter)
             if (!DynamoModel.IsTestMode && !HomeSpace.HasUnsavedChanges && (currentWorkspaceViewModel?.IsHomeSpace ?? true) && HomeSpace.HasRunWithoutCrash)
             {
                 // Currently IsDNADataIngestionPipelineinBeta is set to true for debug builds and false for release builds. so we need to modify it here, when testing the pipeline in debug builds.
-                if (preferencesViewModel.IsMLAutocompleteTOUApproved && !IsDNADataIngestionPipelineinBeta && Model.CurrentWorkspace.IsValidForFDX && currentWorkspaceViewModel.Checksum != string.Empty)
+                if (!IsDNADataIngestionPipelineinBeta && Model.CurrentWorkspace.IsValidForFDX && currentWorkspaceViewModel.Checksum != string.Empty)
                 {
                     if (HasDifferentialCheckSum())
                     {

src/DynamoMLDataPipeline/DynamoMLDataPipeline.cs

@@ -296,6 +296,11 @@ void SendToMLDataPipeline(string filePath, RestClient client, string token)
 
         public void SendWorkspaceLog(string filePath)
         {
+            if (!DynamoModel.PreferenceSettings.IsMLAutocompleteTOUApproved)
+            {
+                throw new InvalidOperationException("User has opted out of data collection, this function shouldn't be called.");
+            }
+
             // Depending on whether we are using Stage or Prod, a token needs to be retrieved 
             // using client and secret id from the app created in the respective environment.
             var token = GetAuthorizationToken();