Skip to content

Commit 3f90564

Browse files
authored
Add telemetry for method modeling panel (#3041)
1 parent b47c561 commit 3f90564

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

extensions/ql-vscode/src/model-editor/method-modeling/method-modeling-view-provider.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -137,6 +137,9 @@ export class MethodModelingViewProvider extends AbstractWebviewViewProvider<
137137
}
138138
case "revealInModelEditor":
139139
await this.revealInModelEditor(msg.method);
140+
void telemetryListener?.sendUIInteraction(
141+
"method-modeling-reveal-in-model-editor",
142+
);
140143

141144
break;
142145

extensions/ql-vscode/src/view/method-modeling/MultipleModeledMethodsPanel.tsx

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ import { Codicon } from "../common";
1313
import { validateModeledMethods } from "../../model-editor/shared/validation";
1414
import { ModeledMethodAlert } from "./ModeledMethodAlert";
1515
import { QueryLanguage } from "../../common/query-language";
16+
import { sendTelemetry } from "../common/telemetry";
1617

1718
export type MultipleModeledMethodsPanelProps = {
1819
language: QueryLanguage;
@@ -81,9 +82,11 @@ export const MultipleModeledMethodsPanel = ({
8182

8283
const handlePreviousClick = useCallback(() => {
8384
setSelectedIndex((previousIndex) => previousIndex - 1);
85+
sendTelemetry("method-modeling-previous-modeling");
8486
}, []);
8587
const handleNextClick = useCallback(() => {
8688
setSelectedIndex((previousIndex) => previousIndex + 1);
89+
sendTelemetry("method-modeling-next-modeling");
8790
}, []);
8891

8992
const validationErrors = useMemo(
@@ -109,6 +112,7 @@ export const MultipleModeledMethodsPanel = ({
109112

110113
onChange(method.signature, newModeledMethods);
111114
selectNewMethod.current = newModeledMethods.length - 1;
115+
sendTelemetry("method-modeling-add-model");
112116
}, [onChange, modeledMethods, method]);
113117

114118
const handleRemoveClick = useCallback(() => {
@@ -123,6 +127,7 @@ export const MultipleModeledMethodsPanel = ({
123127

124128
onChange(method.signature, newModeledMethods);
125129
setSelectedIndex(newSelectedIndex);
130+
sendTelemetry("method-modeling-remove-model");
126131
}, [onChange, modeledMethods, selectedIndex, method]);
127132

128133
const handleChange = useCallback(

0 commit comments

Comments
 (0)