From 76964355043ac7bfd455c69482a8d233bf1053cc Mon Sep 17 00:00:00 2001
From: 18alantom <2.alan.tom@gmail.com>
Date: Wed, 9 Mar 2022 16:49:26 +0530
Subject: [PATCH] chore: add few more logs

---
 frappe/index.js                   |  1 +
 frappe/model/document.js          |  5 +++++
 src/main.js                       |  1 -
 src/pages/PrintView/PrintView.vue | 11 +++++++----
 src/telemetry/types.ts            | 14 ++++++++++----
 5 files changed, 23 insertions(+), 9 deletions(-)

diff --git a/frappe/index.js b/frappe/index.js
index 81560ad5..dae3e469 100644
--- a/frappe/index.js
+++ b/frappe/index.js
@@ -383,4 +383,5 @@ module.exports = {
   },
   t,
   T,
+  store: {},
 };
diff --git a/frappe/model/document.js b/frappe/model/document.js
index 2fe9cde0..d9ca4b5a 100644
--- a/frappe/model/document.js
+++ b/frappe/model/document.js
@@ -3,6 +3,8 @@ const Observable = require('frappe/utils/observable');
 const naming = require('./naming');
 const { isPesa } = require('../utils/index');
 const { DEFAULT_INTERNAL_PRECISION } = require('../utils/consts');
+const telemetry = require('@/telemetry/telemetry');
+const { Verb } = require('@/telemetry/types');
 
 module.exports = class BaseDocument extends Observable {
   constructor(data) {
@@ -577,6 +579,7 @@ module.exports = class BaseDocument extends Observable {
     await this.trigger('afterInsert');
     await this.trigger('afterSave');
 
+    telemetry.log(Verb.Created, this.doctype);
     return this;
   }
 
@@ -620,6 +623,8 @@ module.exports = class BaseDocument extends Observable {
     await this.trigger('beforeDelete');
     await frappe.db.delete(this.doctype, this.name);
     await this.trigger('afterDelete');
+
+    telemetry.log(Verb.Deleted, this.doctype);
   }
 
   async submitOrRevert(isSubmit) {
diff --git a/src/main.js b/src/main.js
index 125e1eff..90d5ea8a 100644
--- a/src/main.js
+++ b/src/main.js
@@ -25,7 +25,6 @@ import { setLanguageMap, showToast, stringifyCircular } from './utils';
   ipcRenderer.invoke = getErrorHandled(ipcRenderer.invoke);
 
   window.frappe = frappe;
-  window.frappe.store = {};
 
   window.onerror = (message, source, lineno, colno, error) => {
     error = error ?? new Error('triggered in window.onerror');
diff --git a/src/pages/PrintView/PrintView.vue b/src/pages/PrintView/PrintView.vue
index 6a8b4072..eec4f7bb 100644
--- a/src/pages/PrintView/PrintView.vue
+++ b/src/pages/PrintView/PrintView.vue
@@ -51,15 +51,17 @@
   </div>
 </template>
 <script>
-import frappe from 'frappe';
+import BackLink from '@/components/BackLink';
+import Button from '@/components/Button';
 import PageHeader from '@/components/PageHeader';
 import SearchBar from '@/components/SearchBar';
-import Button from '@/components/Button';
-import BackLink from '@/components/BackLink';
 import TwoColumnForm from '@/components/TwoColumnForm';
+import { IPC_ACTIONS } from '@/messages';
+import telemetry from '@/telemetry/telemetry';
+import { Verb } from '@/telemetry/types';
 import { makePDF } from '@/utils';
 import { ipcRenderer } from 'electron';
-import { IPC_ACTIONS } from '@/messages';
+import frappe from 'frappe';
 
 export default {
   name: 'PrintView',
@@ -96,6 +98,7 @@ export default {
       if (!savePath) return;
 
       const html = this.$refs.printContainer.innerHTML;
+      telemetry.log(Verb.Exported, 'SalesInvoice', { extension: 'pdf' });
       makePDF(html, savePath);
     },
     async getSavePath() {
diff --git a/src/telemetry/types.ts b/src/telemetry/types.ts
index d35a8a90..60cb6d04 100644
--- a/src/telemetry/types.ts
+++ b/src/telemetry/types.ts
@@ -1,5 +1,6 @@
 import { DoctypeName } from 'models/types';
 
+export type AppVersion = string;
 export type UniqueId = string;
 export type Timestamp = number;
 
@@ -27,14 +28,19 @@ export interface Telemetry {
   timeline?: InteractionEvent[];
   counts?: Count;
   locale: Locale;
+  version: AppVersion;
 }
 
 export enum Verb {
-  Saved = 'saved',
-  Submitted = 'sumbitted',
-  Canceled = 'canceled',
+  Created = 'created',
   Deleted = 'deleted',
   Navigated = 'navigated',
+  Imported = 'imported',
+  Exported = 'exported',
 }
 
-export enum Noun {}
+export enum NounEnum {
+  Route = 'route',
+}
+
+export type Noun = string | NounEnum;