fix(editor): Add telemetry to resource moving (#9720)

n8n-io · Jun 14, 2024 · e84d253 · e84d253
1 parent d62ca78
commit e84d253
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 3 deletions.
diff --git a/packages/editor-ui/src/components/CredentialCard.vue b/packages/editor-ui/src/components/CredentialCard.vue
@@ -23,7 +23,7 @@ const CREDENTIAL_LIST_ITEM_ACTIONS = {
 const props = withDefaults(
 	defineProps<{
 		data: ICredentialsResponse;
-		readOnly: boolean;
+		readOnly?: boolean;
 	}>(),
 	{
 		data: () => ({

diff --git a/packages/editor-ui/src/components/Projects/ProjectMoveResourceConfirmModal.vue b/packages/editor-ui/src/components/Projects/ProjectMoveResourceConfirmModal.vue
@@ -7,6 +7,7 @@ import { useProjectsStore } from '@/stores/projects.store';
 import Modal from '@/components/Modal.vue';
 import { N8nCheckbox, N8nText } from 'n8n-design-system';
 import { useToast } from '@/composables/useToast';
+import { useTelemetry } from '@/composables/useTelemetry';
 
 const props = defineProps<{
 	modalName: string;
@@ -21,6 +22,7 @@ const i18n = useI18n();
 const toast = useToast();
 const uiStore = useUIStore();
 const projectsStore = useProjectsStore();
+const telemetry = useTelemetry();
 
 const checks = ref([false, false]);
 const allChecked = computed(() => checks.value.every(Boolean));
@@ -43,6 +45,10 @@ const confirm = async () => {
 			props.data.projectId,
 		);
 		closeModal();
+		telemetry.track(`User successfully moved ${props.data.resourceType}`, {
+			[`${props.data.resourceType}_id`]: props.data.resource.id,
+			project_from_type: projectsStore.currentProject?.type ?? projectsStore.personalProject?.type,
+		});
 	} catch (error) {
 		toast.showError(
 			error.message,

diff --git a/packages/editor-ui/src/components/Projects/ProjectMoveResourceModal.vue b/packages/editor-ui/src/components/Projects/ProjectMoveResourceModal.vue
@@ -1,12 +1,13 @@
 <script lang="ts" setup>
-import { ref, computed } from 'vue';
+import { ref, computed, onMounted } from 'vue';
 import type { ICredentialsResponse, IWorkflowDb } from '@/Interface';
 import { useI18n } from '@/composables/useI18n';
 import { useUIStore } from '@/stores/ui.store';
 import { useProjectsStore } from '@/stores/projects.store';
 import Modal from '@/components/Modal.vue';
 import { PROJECT_MOVE_RESOURCE_CONFIRM_MODAL } from '@/constants';
 import { splitName } from '@/utils/projects.utils';
+import { useTelemetry } from '@/composables/useTelemetry';
 
 const props = defineProps<{
 	modalName: string;
@@ -19,6 +20,7 @@ const props = defineProps<{
 const i18n = useI18n();
 const uiStore = useUIStore();
 const projectsStore = useProjectsStore();
+const telemetry = useTelemetry();
 
 const projectId = ref<string | null>(null);
 const processedName = computed(() => {
@@ -48,6 +50,13 @@ const next = () => {
 		},
 	});
 };
+
+onMounted(() => {
+	telemetry.track(`User clicked to move a ${props.data.resourceType}`, {
+		[`${props.data.resourceType}_id`]: props.data.resource.id,
+		project_from_type: projectsStore.currentProject?.type ?? projectsStore.personalProject?.type,
+	});
+});
 </script>
 <template>
 	<Modal width="500px" :name="props.modalName" data-test-id="project-move-resource-modal">

diff --git a/packages/editor-ui/src/components/WorkflowCard.vue b/packages/editor-ui/src/components/WorkflowCard.vue
@@ -36,7 +36,7 @@ const WORKFLOW_LIST_ITEM_ACTIONS = {
 const props = withDefaults(
 	defineProps<{
 		data: IWorkflowDb;
-		readOnly: boolean;
+		readOnly?: boolean;
 	}>(),
 	{
 		data: () => ({