enhance(telemetry): enable Glean automatic page load events (#10985)

chore(telemetry): remove unnecessary `max_events` option fix(telemetry): allow live debugging Removes the `max_events` option (we already default to 1 in recent versions). and changes the location of the logging enabled/disabled so that live debugging can happen without the logging being overwritten. Co-authored-by: Alessio Placitelli <[email protected]>
mdn · Apr 24, 2024 · 3b8e023 · 3b8e023
1 parent 223fd3c
commit 3b8e023
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/client/src/telemetry/glean-context.tsx b/client/src/telemetry/glean-context.tsx
@@ -91,7 +91,7 @@ function glean(): GleanAnalytics {
   const uploadEnabled = !userIsOptedOut && GLEAN_ENABLED;
 
   Glean.initialize(GLEAN_APP_ID, uploadEnabled, {
-    maxEvents: 1,
+    enableAutoPageLoadEvents: true,
     channel: GLEAN_CHANNEL,
     migrateFromLegacyStorage: true,
     serverEndpoint: DEV_MODE
@@ -101,8 +101,8 @@ function glean(): GleanAnalytics {
 
   if (DEV_MODE) {
     Glean.setDebugViewTag("mdn-dev");
+    Glean.setLogPings(GLEAN_DEBUG);
   }
-  Glean.setLogPings(GLEAN_DEBUG);
 
   const gleanContext = {
     page: (page: PageProps) => {