add bio and nickname

Hanagotchi · Apr 3, 2024 · fb3b117 · fb3b117
1 parent e50ffb7
commit fb3b117
Show file tree

Hide file tree

Showing 6 changed files with 28 additions and 8 deletions.
diff --git a/app/docker/tablas.sql b/app/docker/tablas.sql
@@ -7,7 +7,9 @@ CREATE TABLE IF NOT EXISTS users_service.users (
     gender VARCHAR(20),
     photo VARCHAR(255),
     birthdate DATE,
-    location JSONB
+    location JSONB,
+    nickname VARCHAR(30),
+    biography VARCHAR(255)
 );
 
 INSERT INTO

diff --git a/app/exceptions/UserException.py b/app/exceptions/UserException.py
@@ -7,8 +7,11 @@ def __init__(self, id: int):
         detail = f"User with id {id} not found"
         super().__init__(status_code=status_code, detail=detail)
 
+
 class InvalidData(HTTPException):
     def __init__(self):
         status_code = status.HTTP_400_BAD_REQUEST
-        detail = f"Invalid user data was provided"
-        super().__init__(status_code=status_code, detail=detail)
+        super().__init__(
+            status_code=status_code,
+            detail="Invalid user data was provided"
+        )
diff --git a/app/models/users.py b/app/models/users.py
@@ -15,3 +15,5 @@ class User(Base):
     photo = Column(String, nullable=True)
     birthdate = Column(Date, nullable=True)
     location = Column(JSON, nullable=True)
+    nickname = Column(String, nullable=True)
+    biography = Column(String, nullable=True)
diff --git a/app/repository/Users.py b/app/repository/Users.py
@@ -49,6 +49,8 @@ def create_user(
         name: Optional[str] = None,
         gender: Optional[str] = None,
         photo: Optional[str] = None,
+        nickname: Optional[str] = None,
+        biography: Optional[str] = None,
         location: Optional[dict] = None,
         birthdate: Optional[date] = None
     ) -> User:
@@ -64,6 +66,10 @@ def create_user(
             user_data["location"] = location
         if birthdate is not None:
             user_data["birthdate"] = birthdate
+        if nickname is not None:
+            user_data["nickname"] = nickname
+        if biography is not None:
+            user_data["biography"] = biography
 
         new_user = User(**user_data)
         self.session.add(new_user)

diff --git a/app/schemas/Schemas.py b/app/schemas/Schemas.py
@@ -11,6 +11,8 @@ class UserSchema(BaseModel):
     photo: str
     birthdate: date
     location: Dict
+    nickname: str
+    biography: str
 
 
 class CreateUserSchema(BaseModel):
@@ -20,6 +22,8 @@ class CreateUserSchema(BaseModel):
     photo: Optional[str] = None
     birthdate: Optional[date] = None
     location: Optional[Dict] = None
+    nickname: Optional[str] = None
+    biography: Optional[str] = None
 
 
 class LoginRequest(BaseModel):
@@ -30,3 +34,7 @@ class UpdateUserSchema(BaseModel):
     name: Optional[str] = None
     gender: Optional[str] = None
     photo: Optional[HttpUrl] = None
+    birthdate: Optional[date] = None
+    location: Optional[Dict] = None
+    nickname: Optional[str] = None
+    biography: Optional[str] = None
diff --git a/app/service/Users.py b/app/service/Users.py
@@ -20,8 +20,7 @@ def get_all_users(self):
         return self.user_repository.get_all_users()
 
     def create_user(self, user_data: dict):
-        email = user_data.get("email")
-        if (not self._validate_location(user_data.get("location"))):
+        if not self._validate_location(user_data.get("location")):
             raise InvalidData()
         return self.user_repository.create_user(**user_data)
 
@@ -81,7 +80,7 @@ def _get_user_info(self, access_token):
 
     def _validate_location(self, location):
         if "lat" in location and "long" in location:
-            if (-90<=location["lat"]<= 90 and -180<=location["long"]<= 180): return True
+            if -90 <= location["lat"] <= 90 and \
+               -180 <= location["long"] <= 180:
+                return True
         return False
-
-