Merge pull request #6727 from hotosm/fastapi-refactor

fix: User email sign up and changed size of database connection pool …
hotosm · Feb 3, 2025 · d7ca3e9 · d7ca3e9
2 parents cf8546a + f771d20
commit d7ca3e9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/backend/db.py b/backend/db.py
@@ -10,7 +10,7 @@ class DatabaseConnection:
 
     def __init__(self):
         self.database = Database(
-            settings.SQLALCHEMY_DATABASE_URI.unicode_string(), min_size=5, max_size=20
+            settings.SQLALCHEMY_DATABASE_URI.unicode_string(), min_size=4, max_size=8
         )
 
     async def connect(self):

diff --git a/backend/services/users/user_service.py b/backend/services/users/user_service.py
@@ -969,8 +969,9 @@ async def register_user_with_email(user_dto: UserRegisterEmailDTO, db: Database)
         if user is None:
             user = UserEmail(email=user_email)
             user = await user.create(db)
-
-        return user
+            return user
+        else:
+            return user.id
 
     @staticmethod
     async def get_interests(user: User, db: Database) -> InterestsListDTO: