pr feedback

immich-app · Nov 26, 2024 · 6186cd3 · 6186cd3
1 parent 769bff3
commit 6186cd3
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 8 deletions.
diff --git a/mobile/lib/providers/auth.provider.dart b/mobile/lib/providers/auth.provider.dart
@@ -1,4 +1,4 @@
-import 'package:flutter/material.dart';
+import 'package:flutter/foundation.dart';
 import 'package:flutter_udid/flutter_udid.dart';
 import 'package:hooks_riverpod/hooks_riverpod.dart';
 import 'package:immich_mobile/entities/store.entity.dart';
@@ -119,9 +119,12 @@ class AuthNotifier extends StateNotifier<AuthState> {
         error,
         stackTrace,
       );
-      debugPrint(
-        "Error getting user information from the server [CATCH ALL] $error $stackTrace",
-      );
+
+      if (kDebugMode) {
+        debugPrint(
+          "Error getting user information from the server [CATCH ALL] $error $stackTrace",
+        );
+      }
     }
 
     // If the user information is successfully retrieved, update the store

diff --git a/mobile/lib/repositories/auth.repository.dart b/mobile/lib/repositories/auth.repository.dart
@@ -16,8 +16,8 @@ class AuthRepository extends DatabaseRepository implements IAuthRepository {
   AuthRepository(super.db);
 
   @override
-  Future<void> clearLocalData() async {
-    await db.writeTxn(() async {
+  Future<void> clearLocalData() {
+    return db.writeTxn(() async {
       await db.assets.clear();
       await db.exifInfos.clear();
       await db.albums.clear();

diff --git a/mobile/lib/services/auth.service.dart b/mobile/lib/services/auth.service.dart
@@ -76,8 +76,8 @@ class AuthService {
   /// - Asset ETag
   ///
   /// All deletions are executed in parallel using [Future.wait].
-  Future<void> clearLocalData() async {
-    await Future.wait([
+  Future<void> clearLocalData() {
+    return Future.wait([
       _authRepository.clearLocalData(),
       Store.delete(StoreKey.currentUser),
       Store.delete(StoreKey.accessToken),