Refactor OrgChangeNotifier and Copilot Notification

microsoft · Sep 30, 2024 · bac7eed · bac7eed
1 parent 805dda8
commit bac7eed
Showing 1 changed file with 19 additions and 20 deletions.
diff --git a/src/client/extension.ts b/src/client/extension.ts
@@ -187,9 +187,6 @@ export async function activate(
     _context.subscriptions.push(cli);
     _context.subscriptions.push(pacTerminal);
 
-    // Init OrgChangeNotifier instance
-    OrgChangeNotifier.createOrgChangeNotifierInstance(pacTerminal.getWrapper());
-
     let copilotNotificationShown = false;
 
     const workspaceFolders =
@@ -234,24 +231,23 @@ export async function activate(
             }
 
             if (!copilotNotificationShown) {
-                if (workspaceContainsPortalConfigFolder(workspaceFolders)) {
-                    let telemetryData = '';
-                    let listOfActivePortals = [];
-                    try {
-                        listOfActivePortals = getPortalsOrgURLs(workspaceFolders, _telemetry);
-                        telemetryData = JSON.stringify(listOfActivePortals);
-                        _telemetry.sendTelemetryEvent("VscodeDesktopUsage", { listOfActivePortals: telemetryData, countOfActivePortals: listOfActivePortals.length.toString() });
-                        oneDSLoggerWrapper.getLogger().traceInfo("VscodeDesktopUsage", { listOfActivePortals: telemetryData, countOfActivePortals: listOfActivePortals.length.toString() });
-                    } catch (exception) {
-                        const exceptionError = exception as Error;
-                        _telemetry.sendTelemetryException(exceptionError, { eventName: 'VscodeDesktopUsage' });
-                        oneDSLoggerWrapper.getLogger().traceError(exceptionError.name, exceptionError.message, exceptionError, { eventName: 'VscodeDesktopUsage' });
-                    }
-
-                    // Show Copilot notification after ECS initialization and workspace check
-                    showNotificationForCopilot(_telemetry, telemetryData, listOfActivePortals.length.toString());
-                    copilotNotificationShown = true;
+                let telemetryData = '';
+                let listOfActivePortals = [];
+                try {
+                    listOfActivePortals = getPortalsOrgURLs(workspaceFolders, _telemetry);
+                    telemetryData = JSON.stringify(listOfActivePortals);
+                    _telemetry.sendTelemetryEvent("VscodeDesktopUsage", { listOfActivePortals: telemetryData, countOfActivePortals: listOfActivePortals.length.toString() });
+                    oneDSLoggerWrapper.getLogger().traceInfo("VscodeDesktopUsage", { listOfActivePortals: telemetryData, countOfActivePortals: listOfActivePortals.length.toString() });
+                } catch (exception) {
+                    const exceptionError = exception as Error;
+                    _telemetry.sendTelemetryException(exceptionError, { eventName: 'VscodeDesktopUsage' });
+                    oneDSLoggerWrapper.getLogger().traceError(exceptionError.name, exceptionError.message, exceptionError, { eventName: 'VscodeDesktopUsage' });
                 }
+
+                // Show Copilot notification after ECS initialization and workspace check
+                showNotificationForCopilot(_telemetry, telemetryData, listOfActivePortals.length.toString());
+                copilotNotificationShown = true;
+
             }
 
         })
@@ -261,6 +257,9 @@ export async function activate(
 
     if (workspaceContainsPortalConfigFolder(workspaceFolders)) {
 
+        // Init OrgChangeNotifier instance
+        OrgChangeNotifier.createOrgChangeNotifierInstance(pacTerminal.getWrapper());
+
         vscode.workspace.onDidOpenTextDocument(didOpenTextDocument);
         vscode.workspace.textDocuments.forEach(didOpenTextDocument);