Dev/ianhu/telemetry on shift enter (#4637)

Add telemetry for the shift enter command

Author: IanMatthewHuff

news/2 Fixes/4636.md

@@ -0,0 +1 @@
+Provide telemetry for when we show the shift-enter banner and if the user clicks yes

src/client/datascience/constants.ts

@@ -103,8 +103,10 @@ export enum Telemetry {
     SubmitCellThroughInput = 'DATASCIENCE.SUBMITCELLFROMREPL',
     ConnectLocalJupyter = 'DATASCIENCE.CONNECTLOCALJUPYTER',
     ConnectRemoteJupyter = 'DATASCIENCE.CONNECTREMOTEJUPYTER',
-    ConnectFailedJupyter = 'DATASCIENCE.CONNECTFAILEDJUPYTER'
-}
+    ConnectFailedJupyter = 'DATASCIENCE.CONNECTFAILEDJUPYTER',
+    ShiftEnterBannerShown = 'DATASCIENCE.SHIFTENTER_BANNER_SHOWN',
+    EnableInteractiveShiftEnter = 'DATASCIENCE.ENABLE_INTERACTIVE_SHIFT_ENTER'
+ }
 
 export namespace HelpLinks {
     export const PythonInteractiveHelpLink = 'https://aka.ms/pyaiinstall';

src/client/datascience/shiftEnterBanner.ts

@@ -10,6 +10,8 @@ import '../common/extensions';
 import { IConfigurationService, IPersistentStateFactory,
     IPythonExtensionBanner } from '../common/types';
 import * as localize from '../common/utils/localize';
+import { captureTelemetry, sendTelemetryEvent } from '../telemetry';
+import { Telemetry } from './constants';
 import { IJupyterExecution } from './types';
 
 export enum InteractiveShiftEnterStateKeys {
@@ -71,6 +73,7 @@ export class InteractiveShiftEnterBanner implements IPythonExtensionBanner {
             return;
         }
 
+        sendTelemetryEvent(Telemetry.ShiftEnterBannerShown);
         const response = await this.appShell.showInformationMessage(this.bannerMessage, ...this.bannerLabels);
         switch (response) {
             case this.bannerLabels[InteractiveShiftEnterLabelIndex.Yes]: {
@@ -98,6 +101,7 @@ export class InteractiveShiftEnterBanner implements IPythonExtensionBanner {
         await this.persistentState.createGlobalPersistentState<boolean>(InteractiveShiftEnterStateKeys.ShowBanner, false).updateValue(false);
     }
 
+    @captureTelemetry(Telemetry.EnableInteractiveShiftEnter)
     public async enableInteractiveShiftEnter(): Promise<void> {
         await this.configuration.updateSetting('dataScience.sendSelectionToInteractiveWindow', true, undefined, ConfigurationTarget.Global);
         await this.disable();

src/client/telemetry/index.ts

@@ -307,6 +307,7 @@ interface IEventNamePropertyMapping {
     [Telemetry.ConnectRemoteJupyter]: never | undefined;
     [Telemetry.DeleteAllCells]: never | undefined;
     [Telemetry.DeleteCell]: never | undefined;
+    [Telemetry.EnableInteractiveShiftEnter]: never | undefined;
     [Telemetry.ExpandAll]: never | undefined;
     [Telemetry.ExportNotebook]: never | undefined;
     [Telemetry.ExportPythonFile]: never | undefined;
@@ -324,6 +325,7 @@ interface IEventNamePropertyMapping {
     [Telemetry.SelectJupyterURI]: never | undefined;
     [Telemetry.SetJupyterURIToLocal]: never | undefined;
     [Telemetry.SetJupyterURIToUserSpecified]: never | undefined;
+    [Telemetry.ShiftEnterBannerShown]: never | undefined;
     [Telemetry.ShowHistoryPane]: never | undefined;
     [Telemetry.StartJupyter]: never | undefined;
     [Telemetry.SubmitCellThroughInput]: never | undefined;