From 37877d5336b8ba487958b5bb83df4b032d92dd1e Mon Sep 17 00:00:00 2001
From: Vishesh Handa <me@vhanda.in>
Date: Mon, 10 May 2021 12:07:49 +0200
Subject: [PATCH] Disable analytics on Desktop

firebase isn't available there, and this just results in more errors.
---
 lib/analytics/analytics.dart | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/lib/analytics/analytics.dart b/lib/analytics/analytics.dart
index dc798d94..e8fc712f 100644
--- a/lib/analytics/analytics.dart
+++ b/lib/analytics/analytics.dart
@@ -1,5 +1,7 @@
 // @dart=2.9
 
+import 'dart:io' show Platform;
+
 import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 
@@ -149,22 +151,30 @@ class Analytics {
     Map<String, String> parameters = const {},
   }) async {
     String name = _eventToString(e);
-    await firebase.logEvent(name: name, parameters: parameters);
+    if (Platform.isAndroid || Platform.isIOS) {
+      await firebase.logEvent(name: name, parameters: parameters);
+    }
     captureErrorBreadcrumb(name: name, parameters: parameters);
   }
 
   Future<void> setAnalyticsCollectionEnabled(bool enabled) async {
     this.enabled = enabled;
-    return firebase.setAnalyticsCollectionEnabled(enabled);
+    if (Platform.isAndroid || Platform.isIOS) {
+      await firebase.setAnalyticsCollectionEnabled(enabled);
+    }
   }
 
   Future<void> setCurrentScreen({@required String screenName}) async {
-    await firebase.setCurrentScreen(screenName: screenName);
+    if (Platform.isAndroid || Platform.isIOS) {
+      await firebase.setCurrentScreen(screenName: screenName);
+    }
   }
 
   Future<void> setUserProperty(
       {@required String name, @required String value}) async {
-    await firebase.setUserProperty(name: name, value: value);
+    if (Platform.isAndroid || Platform.isIOS) {
+      await firebase.setUserProperty(name: name, value: value);
+    }
   }
 }