feat: implement contact form submission endpoint with validation

endpoints/email.post.py

@@ -15,29 +15,24 @@ async def submit_contact_form(
     Submit a new contact form with name, email, and message.
     All fields are required and email must be in valid format.
     """
-    # Validate the form data
+    # Validate the contact form data
-    errors = validate_contact_form_data({
+    validation_errors = validate_contact_form_data(contact_form.dict())
-        "name": contact_form.name,
-        "email": contact_form.email,
-        "message": contact_form.message
-    })
     
-    # If there are validation errors, return 400 with error details
+    # If there are validation errors, return a 400 response with the errors
-    if errors:
+    if validation_errors:
         raise HTTPException(
             status_code=status.HTTP_400_BAD_REQUEST,
-            detail=errors
+            detail=validation_errors
         )
     
-    # Create the contact form in the database
+    # Create the contact form submission
-    created_form = create_contact_form(db=db, contact_form_data=contact_form)
+    new_contact_form = create_contact_form(db=db, contact_form_data=contact_form)
     
     return {
-        "status": "success",
+        "id": new_contact_form.id,
-        "message": "Contact form submitted successfully",
+        "name": new_contact_form.name,
-        "data": {
+        "email": new_contact_form.email,
-            "id": created_form.id,
+        "message": new_contact_form.message,
-            "name": created_form.name,
+        "created_at": new_contact_form.created_at,
-            "email": created_form.email
+        "status": "success"
-        }
     }

helpers/contact_form_helpers.py

@@ -45,21 +45,27 @@ def validate_contact_form_data(data: Dict[str, Any]) -> Dict[str, str]:
     """
     errors = {}
     
-    # Check required fields
+    # Check required fields with specific error messages
     if not data.get("name"):
         errors["name"] = "Name is required"
+    elif not isinstance(data.get("name"), str) or len(data.get("name").strip()) == 0:
+        errors["name"] = "Name must be a non-empty string"
     
+    # Email validation with detailed error handling
     if not data.get("email"):
         errors["email"] = "Email is required"
-    elif data.get("email"):
+    else:
         try:
             # Validate email format using email_validator package
-            validate_email(data["email"])
+            validate_email(data["email"], check_deliverability=True)
         except EmailNotValidError as e:
-            errors["email"] = f"Email validation failed: {str(e)}"
+            errors["email"] = f"Invalid email format: {str(e)}"
     
+    # Message validation
     if not data.get("message"):
         errors["message"] = "Message is required"
+    elif not isinstance(data.get("message"), str) or len(data.get("message").strip()) == 0:
+        errors["message"] = "Message must be a non-empty string"
     
     return errors