Just use User model

We don't actually need separate models here as the `User` with events isn't used at all. Simplify!

Author: badlydrawnrob

building-with-fast-api/chapter_05/models/users.py

@@ -5,9 +5,9 @@
 # ------------------------------------------------------------------------------
 # Our USER model
 # ==============================================================================
-# Here we'll set our user model, which has a `User.Events` column, to list all
+# ~~Here we'll set our user model, which has a `User.Events` column, to list all
 # the events attached to that specific user. We have a couple of types, depending
-# on the situation (for example, sign-in form). We could also create a separate
+# on the situation (for example, sign-in form).~~ We could also create a separate
 # `User` model for the `response_model=` in our routes, to hide the `password`.
 #
 # Questions
@@ -19,9 +19,3 @@
 class User(BaseModel):
     email: EmailStr # A custom type for emails
     password: str
-    events: Optional[List[Event]] # Empty by default, currently not used
-
-# For `sign-up` and `sign-in`
-class UserSign(BaseModel):
-    email: EmailStr
-    password: str

building-with-fast-api/chapter_05/routes/users.py

@@ -1,5 +1,5 @@
 from fastapi import APIRouter, HTTPException, status
-from models.users import User, UserSign
+from models.users import User
 
 # ------------------------------------------------------------------------------
 # Our USERS routes
@@ -27,15 +27,15 @@
 # for both, rather than creating separate models for `sign-up` and `sign-in`.
 
 @user_router.post("/signup")
-async def sign_new_user(data: UserSign) -> dict:
+async def sign_new_user(data: User) -> dict:
     if data.email in users:
         raise HTTPException(status_code=409, detail="Username already exists")
 
     users[data.email] = data # != Ask copilot to explain this line
     return { "message": "User registered!" }
 
 @user_router.post("/signin")
-async def sign_in_user(user: UserSign) -> dict:
+async def sign_in_user(user: User) -> dict:
     # Check if `UserSign().email` is in the `users` dictionary
     # You could also use `if _ or _` here!
     if user.email not in users: