Merge pull request #62 from owlistic-notes/user-profile-fixes

Clear preferences on logout

Author: DavideRutigliano

src/backend/services/auth_service.go

@@ -42,12 +42,6 @@ func (s *AuthService) Login(db *database.Database, email, password string) (stri
 		return "", ErrInvalidCredentials
 	}
 
-	// Check if the user has any notebooks
-	var notebookCount int64
-	if err := db.DB.Model(&models.Notebook{}).Where("user_id = ?", user.ID).Count(&notebookCount).Error; err != nil {
-		return "", err
-	}
-
 	// Use the utility function instead
 	tokenString, err := token.GenerateToken(user.ID, user.Email, s.jwtSecret, s.jwtExpiration)
 	if err != nil {

src/frontend/lib/services/auth_service.dart

@@ -230,23 +230,15 @@ class AuthService extends BaseService {
 
   Future<bool> logout() async {
     try {
-      // Call logout endpoint if it exists and we have a token
-      if (isLoggedIn) {
-        try {
-          await authenticatedPost('/api/v1/logout', {});
-        } catch (e) {
-          // Just log the error but continue with local logout
-          _logger.error('Error calling logout endpoint', e);
-        }
-      }
-      
       // Clear token regardless of response
       await clearToken();
+      await clearPreferences();
       _logger.info('Logged out successfully');
       return true;
     } catch (e) {
       _logger.error('Error during logout', e);
       await clearToken(); // Still clear token on error
+      await clearPreferences();
       return false;
     }
   }
@@ -424,6 +416,17 @@ Future<String?> getCurrentUserId() async {
       return null;
     }
   }
+
+  Future<void> clearPreferences() async {
+    try {
+      // First try to get from shared preferences for better performance
+      final prefs = await SharedPreferences.getInstance();
+      prefs.clear();
+    } catch (e) {
+      _logger.error('Error getting current user ID', e);
+      return null;
+    }
+  }
 
   // Safe login with better error handling
   Future<bool> loginSafe(String email, String password) async {