Merge pull request #246 from yumemi-inc/fix/unnecessaryCreation

Refactor AppStatus provider to use existing state instances directly
yumemi-inc · Jun 11, 2024 · 5bb4427 · 5bb4427
2 parents d48b723 + 1b30ebd
commit 5bb4427
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 11 deletions.
diff --git a/packages/cores/core/lib/src/app_status/provider/app_status_provider.dart b/packages/cores/core/lib/src/app_status/provider/app_status_provider.dart
@@ -1,6 +1,4 @@
 import 'package:cores_core/src/app_status/model/app_status.dart';
-import 'package:cores_core/src/app_status/model/force_update_status.dart';
-import 'package:cores_core/src/app_status/model/maintenance_mode_status.dart';
 import 'package:cores_core/src/app_status/provider/force_update_provider.dart';
 import 'package:cores_core/src/app_status/provider/maintenance_mode_provider.dart';
 
@@ -10,15 +8,11 @@ part 'app_status_provider.g.dart';
 
 @Riverpod(keepAlive: true)
 AppStatus appStatus(AppStatusRef ref) {
-  final maintenanceModeState = ref.watch(maintenanceModeProvider);
-  final forceUpdateState = ref.watch(forceUpdateProvider);
+  final maintenanceModeStatus = ref.watch(maintenanceModeProvider);
+  final forceUpdateStatus = ref.watch(forceUpdateProvider);
 
   return AppStatus(
-    maintenanceModeStatus: MaintenanceModeStatus(
-      enabled: maintenanceModeState.enabled,
-    ),
-    forceUpdateStatus: ForceUpdateStatus(
-      enabled: forceUpdateState.enabled,
-    ),
+    maintenanceModeStatus: maintenanceModeStatus,
+    forceUpdateStatus: forceUpdateStatus,
   );
 }
diff --git a/packages/cores/core/lib/src/app_status/provider/app_status_provider.g.dart b/packages/cores/core/lib/src/app_status/provider/app_status_provider.g.dart