k

backend/services/auth_service.py

@@ -54,6 +54,9 @@ def register_user(email: str, password: str) -> dict:
                 'message': 'Failed to register user'
             }
     except Exception as e:
+        # Log the full error for debugging
+        current_app.logger.error(f"Registration error for email {email}: {str(e)}")
+        
         # Check if it's a duplicate user error
         error_str = str(e).lower()
         if 'already registered' in error_str or 'already exists' in error_str:
@@ -66,15 +69,17 @@ def register_user(email: str, password: str) -> dict:
                 'success': False,
                 'message': 'Please enter a valid email address.'
             }
-        elif 'password' in error_str:
+        # More specific check for Supabase password policy errors
+        elif 'password' in error_str and ('weak' in error_str or 'policy' in error_str or 'requirement' in error_str):
             return {
                 'success': False,
                 'message': 'Password does not meet requirements. Please use at least 8 characters.'
             }
         else:
+            # For other errors, provide a more generic message to the user but log the details
             return {
                 'success': False,
-                'message': f'Registration failed: {str(e)}'
+                'message': 'Registration failed. Please check your information and try again.'
             }
 
 def login_user(email: str, password: str, remember_me: bool = False) -> dict:

frontend/src/store/reducers/sourcesSlice.js

@@ -77,7 +77,10 @@ const sourcesSlice = createSlice({
       .addCase(addSource.fulfilled, (state, action) => {
         state.loading = false;
         // Add the new source to the list
-        state.items = [...state.items, action.payload];
+        // The payload should contain the full source object, not just a success message
+        if (action.payload && action.payload.source) {
+          state.items = [...state.items, action.payload.source];
+        }
       })
       .addCase(addSource.rejected, (state, action) => {
         state.loading = false;