Merge pull request #509 from provokateurin/fix/hide-useless-push-notifications-popups

fix(neon): Hide useless push notifications popups

packages/neon/neon/lib/src/pages/home.dart

@@ -74,6 +74,7 @@ class _HomePageState extends State<HomePage> {
   @override
   void dispose() {
     unawaited(_versionCheckSubscription.cancel());
+    GlobalPopups().dispose();
     super.dispose();
   }
 

packages/neon/neon/lib/src/utils/global_popups.dart

@@ -1,9 +1,12 @@
+import 'dart:async';
+
 import 'package:flutter/material.dart';
 import 'package:meta/meta.dart';
 import 'package:neon/l10n/localizations.dart';
 import 'package:neon/src/blocs/first_launch.dart';
 import 'package:neon/src/blocs/next_push.dart';
 import 'package:neon/src/pages/settings.dart';
+import 'package:neon/src/platform/platform.dart';
 import 'package:neon/src/router.dart';
 import 'package:neon/src/utils/global_options.dart';
 import 'package:provider/provider.dart';
@@ -22,8 +25,19 @@ class GlobalPopups {
   static GlobalPopups? instance;
 
   bool _registered = false;
+  late BuildContext _context;
+  final _subscriptions = <StreamSubscription>[];
+
+  void dispose() {
+    for (final subscription in _subscriptions) {
+      unawaited(subscription.cancel());
+    }
+    _subscriptions.clear();
+    _registered = false;
+  }
 
   void register(final BuildContext context) {
+    _context = context;
     if (_registered) {
       return;
     }
@@ -31,29 +45,36 @@ class GlobalPopups {
     final globalOptions = Provider.of<GlobalOptions>(context, listen: false);
     final firstLaunchBloc = Provider.of<FirstLaunchBloc>(context, listen: false);
     final nextPushBloc = Provider.of<NextPushBloc>(context, listen: false);
+    final platform = Provider.of<NeonPlatform>(context, listen: false);
 
+    _subscriptions.addAll([
+      if (platform.canUsePushNotifications) ...[
         firstLaunchBloc.onFirstLaunch.listen((final _) {
-      if (globalOptions.pushNotificationsEnabled.enabled) {
-        if (!context.mounted) {
+          assert(context.mounted, 'Context should be mounted');
+          if (!globalOptions.pushNotificationsEnabled.enabled) {
             return;
           }
-        ScaffoldMessenger.of(context).showSnackBar(
+
+          ScaffoldMessenger.of(_context).showSnackBar(
             SnackBar(
-            content: Text(AppLocalizations.of(context).firstLaunchGoToSettingsToEnablePushNotifications),
+              content: Text(AppLocalizations.of(_context).firstLaunchGoToSettingsToEnablePushNotifications),
               action: SnackBarAction(
-              label: AppLocalizations.of(context).settings,
+                label: AppLocalizations.of(_context).settings,
                 onPressed: () {
-                const SettingsRoute(initialCategory: SettingsCageories.pushNotifications).go(context);
+                  const SettingsRoute(initialCategory: SettingsCageories.pushNotifications).go(_context);
                 },
               ),
             ),
           );
+        }),
+        nextPushBloc.onNextPushSupported.listen((final _) async {
+          assert(context.mounted, 'Context should be mounted');
+          if (!globalOptions.pushNotificationsEnabled.enabled) {
+            return;
           }
-    });
 
-    nextPushBloc.onNextPushSupported.listen((final _) async {
           await showDialog(
-        context: context,
+            context: _context,
             builder: (final context) => AlertDialog(
               title: Text(AppLocalizations.of(context).nextPushSupported),
               content: Text(AppLocalizations.of(context).nextPushSupportedText),
@@ -77,7 +98,9 @@ class GlobalPopups {
               ],
             ),
           );
-    });
+        }),
+      ],
+    ]);
 
     _registered = true;
   }