Merge pull request #654 from mjashanks/bugfix

Bugfixes

packages/builder/src/analytics.js

@@ -3,6 +3,8 @@ import posthog from "posthog-js"
 import api from "builderStore/api"
 
 let analyticsEnabled
+const posthogConfigured = process.env.POSTHOG_TOKEN && process.env.POSTHOG_URL
+const sentryConfigured = process.env.SENTRY_DSN
 
 async function activate() {
   if (analyticsEnabled === undefined) {
@@ -13,21 +15,22 @@ async function activate() {
     analyticsEnabled = (await response.json()) === true
   }
   if (!analyticsEnabled) return
-  Sentry.init({ dsn: process.env.SENTRY_DSN })
+  if (sentryConfigured) Sentry.init({ dsn: process.env.SENTRY_DSN })
-  if (!process.env.POSTHOG_TOKEN) return
+  if (posthogConfigured) {
-  posthog.init(process.env.POSTHOG_TOKEN, {
+    posthog.init(process.env.POSTHOG_TOKEN, {
-    api_host: process.env.POSTHOG_URL,
+      api_host: process.env.POSTHOG_URL,
-  })
+    })
-  posthog.set_config({ persistence: "cookie" })
+    posthog.set_config({ persistence: "cookie" })
+  }
 }
 
 function identify(id) {
-  if (!analyticsEnabled) return
+  if (!analyticsEnabled || !id) return
-  if (!id) return
+  if (posthogConfigured) posthog.identify(id)
-  posthog.identify(id)
+  if (sentryConfigured)
-  Sentry.configureScope(scope => {
+    Sentry.configureScope(scope => {
-    scope.setUser({ id: id })
+      scope.setUser({ id: id })
-  })
+    })
 }
 
 async function identifyByApiKey(apiKey) {

packages/builder/src/pages/index.svelte

@@ -37,7 +37,9 @@
     if (keys.userId) {
       hasKey = true
       analytics.identify(keys.userId)
-    } else {
+    }
+
+    if (!keys.budibase) {
       showCreateAppModal()
     }
   }