Merge pull request #13872 from strapi/feature/amplitude-arch

Add tracking property for os.arch() to Amplitude / Sentry (project creation)
strapi · Aug 24, 2022 · 78af3eb · 78af3eb
2 parents cd16049 + 8e14a14
commit 78af3eb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/packages/generators/app/lib/index.js b/packages/generators/app/lib/index.js
@@ -5,8 +5,6 @@ const os = require('os');
 const crypto = require('crypto');
 const { v4: uuidv4 } = require('uuid');
 const sentry = require('@sentry/node');
-// FIXME
-/* eslint-disable import/extensions */
 const hasYarn = require('./utils/has-yarn');
 const checkRequirements = require('./utils/check-requirements');
 const { trackError, captureException } = require('./utils/usage');
@@ -60,6 +58,7 @@ const generateNewApp = (projectDirectory, cliArguments) => {
 
   sentry.configureScope(function scope(sentryScope) {
     const tags = {
+      os_arch: os.arch(),
       os_type: os.type(),
       os_platform: os.platform(),
       os_release: os.release(),

diff --git a/packages/generators/app/lib/utils/usage.js b/packages/generators/app/lib/utils/usage.js
@@ -108,6 +108,7 @@ function trackUsage({ event, scope, error }) {
   const properties = {
     error: typeof error === 'string' ? error : error && error.message,
     os: os.type(),
+    os_arch: os.arch(),
     os_platform: os.platform(),
     os_release: os.release(),
     node_version: process.version,