feat: handle update modal hiding after app update

ice-blockchain · Dec 24, 2024 · 3858775 · 3858775
1 parent 519632a
commit 3858775
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 14 deletions.
diff --git a/lib/app/components/app_update_handler/app_update_handler.dart b/lib/app/components/app_update_handler/app_update_handler.dart
@@ -1,3 +1,5 @@
+// SPDX-License-Identifier: ice License 1.0
+
 import 'package:flutter/material.dart';
 import 'package:hooks_riverpod/hooks_riverpod.dart';
 import 'package:ion/app/components/app_update_handler/hooks/use_app_update.dart';

diff --git a/lib/app/components/app_update_handler/hooks/use_app_update.dart b/lib/app/components/app_update_handler/hooks/use_app_update.dart
@@ -1,18 +1,20 @@
+// SPDX-License-Identifier: ice License 1.0
+
 import 'package:flutter_hooks/flutter_hooks.dart';
 import 'package:hooks_riverpod/hooks_riverpod.dart';
 import 'package:ion/app/features/config/providers/force_update_provider.c.dart';
 
 bool useAppUpdate(WidgetRef ref) {
   final isModalShown = useState(false);
-
   final forceUpdateState = ref.watch(forceUpdateProvider);
 
   useEffect(
     () {
       if (forceUpdateState.showUpdateModal && !isModalShown.value) {
         isModalShown.value = true;
+      } else if (!forceUpdateState.showUpdateModal && isModalShown.value) {
+        isModalShown.value = false;
       }
-
       return null;
     },
     [forceUpdateState],

diff --git a/lib/app/features/config/providers/force_update_provider.c.dart b/lib/app/features/config/providers/force_update_provider.c.dart
@@ -7,7 +7,6 @@ import 'package:freezed_annotation/freezed_annotation.dart';
 import 'package:ion/app/features/config/providers/config_provider.c.dart';
 import 'package:ion/app/features/config/providers/force_update_last_sync_date_provider.c.dart';
 import 'package:ion/app/features/core/providers/app_lifecycle_provider.c.dart';
-import 'package:ion/app/services/logger/logger.dart';
 import 'package:ion/app/utils/force_update.dart';
 import 'package:package_info_plus/package_info_plus.dart';
 import 'package:riverpod_annotation/riverpod_annotation.dart';
@@ -30,6 +29,8 @@ class ForceUpdate extends _$ForceUpdate {
 
   @override
   ForceUpdateState build() {
+    _checkAndUpdateConfig();
+
     ref.listen<AppLifecycleState>(
       appLifecycleProvider,
       (previous, next) {
@@ -45,8 +46,6 @@ class ForceUpdate extends _$ForceUpdate {
   Future<void> _checkAndUpdateConfig() async {
     final lastSyncDate = ref.read(forceUpdateLastSyncDateNotifierProvider);
 
-    Logger.log('lastSyncDate: $lastSyncDate');
-
     if (lastSyncDate == null ||
         DateTime.now().difference(lastSyncDate).inMilliseconds >= eightHoursInMilliseconds) {
       final remoteVersion =
@@ -55,17 +54,15 @@ class ForceUpdate extends _$ForceUpdate {
       final packageInfo = await PackageInfo.fromPlatform();
       final localVersion = packageInfo.version;
 
-      if (ForceUpdateUtil.isVersionOutdated(localVersion, remoteVersion)) {
-        state = state.copyWith(showUpdateModal: true);
-      } else {
+      if (localVersion == remoteVersion) {
         state = state.copyWith(showUpdateModal: false);
-      }
 
-      ref.read(forceUpdateLastSyncDateNotifierProvider.notifier).updateLastSyncDate(DateTime.now());
+        ref
+            .read(forceUpdateLastSyncDateNotifierProvider.notifier)
+            .updateLastSyncDate(DateTime.now());
+      } else if (ForceUpdateUtil.isVersionOutdated(localVersion, remoteVersion)) {
+        state = state.copyWith(showUpdateModal: true);
+      }
     }
   }
-
-  void resetUpdateModal() {
-    state = state.copyWith(showUpdateModal: false);
-  }
 }