mohiit1502/Automata
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

refactor(editor): Add telemetry for workflow history (no-changelog) (#8072)

Browse Source 
https://linear.app/n8n/issue/PAY-1142
This commit is contained in:
Iván Ovejero
2023-12-18 16:10:45 +01:00
committed by GitHub
parent 73d400a1bf
commit 62ce962ec9
2 changed files with 9 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
packages/editor-ui/src/views/WorkflowHistory.vue
View File

@@ -18,6 +18,7 @@ import { useWorkflowHistoryStore } from '@/stores/workflowHistory.store';
import { useUIStore } from '@/stores/ui.store'; import { useUIStore } from '@/stores/ui.store';
import { useWorkflowsStore } from '@/stores/workflows.store'; import { useWorkflowsStore } from '@/stores/workflows.store';
import { telemetry } from '@/plugins/telemetry'; import { telemetry } from '@/plugins/telemetry';
import { useRootStore } from '@/stores/n8nRoot.store';
type WorkflowHistoryActionRecord = { type WorkflowHistoryActionRecord = {
[K in Uppercase<WorkflowHistoryActionTypes[number]>]: Lowercase<K>; [K in Uppercase<WorkflowHistoryActionTypes[number]>]: Lowercase<K>;
@@ -76,6 +77,7 @@ const evaluatedPruneTime = computed(() => Math.floor(workflowHistoryStore.evalua
const sendTelemetry = (event: string) => { const sendTelemetry = (event: string) => {
telemetry.track(event, { telemetry.track(event, {
instance_id: useRootStore().instanceId,
workflow_id: route.params.workflowId, workflow_id: route.params.workflowId,
}); });
}; };

7
packages/editor-ui/src/views/__tests__/WorkflowHistory.test.ts
View File

@@ -107,6 +107,7 @@ describe('WorkflowHistory', () => {
params: { workflowId, versionId: versionData.versionId }, params: { workflowId, versionId: versionData.versionId },
}); });
expect(telemetry.track).toHaveBeenCalledWith('User opened workflow history', { expect(telemetry.track).toHaveBeenCalledWith('User opened workflow history', {
instance_id: '',
workflow_id: workflowId, workflow_id: workflowId,
}); });
}); });
@@ -124,6 +125,7 @@ describe('WorkflowHistory', () => {
await waitFor(() => { await waitFor(() => {
expect(router.replace).not.toHaveBeenCalled(); expect(router.replace).not.toHaveBeenCalled();
expect(telemetry.track).toHaveBeenCalledWith('User selected version', { expect(telemetry.track).toHaveBeenCalledWith('User selected version', {
instance_id: '',
workflow_id: workflowId, workflow_id: workflowId,
}); });
}); });
@@ -142,6 +144,7 @@ describe('WorkflowHistory', () => {
params: { workflowId, versionId }, params: { workflowId, versionId },
}); });
expect(telemetry.track).toHaveBeenCalledWith('User selected version', { expect(telemetry.track).toHaveBeenCalledWith('User selected version', {
instance_id: '',
workflow_id: workflowId, workflow_id: workflowId,
}); });
}); });
@@ -159,6 +162,7 @@ describe('WorkflowHistory', () => {
params: { workflowId, versionId }, params: { workflowId, versionId },
}); });
expect(telemetry.track).toHaveBeenCalledWith('User opened version in new tab', { expect(telemetry.track).toHaveBeenCalledWith('User opened version in new tab', {
instance_id: '',
workflow_id: workflowId, workflow_id: workflowId,
}); });
}); });
@@ -180,6 +184,7 @@ describe('WorkflowHistory', () => {
params: { workflowId, versionId }, params: { workflowId, versionId },
}); });
expect(telemetry.track).toHaveBeenCalledWith('User opened version in new tab', { expect(telemetry.track).toHaveBeenCalledWith('User opened version in new tab', {
instance_id: '',
workflow_id: workflowId, workflow_id: workflowId,
}); });
}); });
@@ -202,6 +207,7 @@ describe('WorkflowHistory', () => {
params: { name: newWorkflowId }, params: { name: newWorkflowId },
}); });
expect(telemetry.track).toHaveBeenCalledWith('User cloned version', { expect(telemetry.track).toHaveBeenCalledWith('User cloned version', {
instance_id: '',
workflow_id: workflowId, workflow_id: workflowId,
}); });
}); });
@@ -217,6 +223,7 @@ describe('WorkflowHistory', () => {
await waitFor(() => { await waitFor(() => {
expect(telemetry.track).toHaveBeenCalledWith('User downloaded version', { expect(telemetry.track).toHaveBeenCalledWith('User downloaded version', {
instance_id: '',
workflow_id: workflowId, workflow_id: workflowId,
}); });
}); });