From 8663cdbd934aa520364a696bf1935a1f1d408b3b Mon Sep 17 00:00:00 2001 From: Jess Kuras Date: Wed, 2 Sep 2026 20:26:53 +0000 Subject: [PATCH] Remove client-side productionBuild guard in AnalyticsImplementation The productionBuild environment variable is not defined during client-side dart2js compilation, causing sendEvent to return early and strip out the dataLayer push. Removing this check matches how dart-lang/site-www implements client analytics and allows client events like inline_feedback to be dispatched to window.dataLayer. --- packages/site_shared/lib/src/analytics/analytics_web.dart | 4 ---- 1 file changed, 4 deletions(-) diff --git a/packages/site_shared/lib/src/analytics/analytics_web.dart b/packages/site_shared/lib/src/analytics/analytics_web.dart index 94e595af88a..dfa0cb5cb7e 100644 --- a/packages/site_shared/lib/src/analytics/analytics_web.dart +++ b/packages/site_shared/lib/src/analytics/analytics_web.dart @@ -6,7 +6,6 @@ import 'package:meta/meta.dart'; import 'package:universal_web/js_interop.dart'; import 'package:universal_web/web.dart' as web; -import '../../util.dart'; import 'analytics.dart'; /// Web implementation of [Analytics]. @@ -16,9 +15,6 @@ import 'analytics.dart'; final class AnalyticsImplementation extends Analytics { @override void sendEvent(String eventName, Map parameters) { - if (!productionBuild) { - return; - } final dataLayer = web.window['dataLayer']; if (dataLayer.isA()) { (dataLayer as JSArray).toDart.add(