feat: Doctor Fees, Email Notifications, and Dynamic Booking

Author: pavanbadempet

backend/appointments.py

@@ -27,6 +27,7 @@ def create_appointment(
 
     new_appt = models.Appointment(
         user_id=current_user.id,
+        doctor_id=appt.doctor_id,
         specialist=appt.specialist,
         date_time=appointment_dt,
         reason=appt.reason,
@@ -36,6 +37,19 @@ def create_appointment(
     db.add(new_appt)
     db.commit()
     db.refresh(new_appt)
+    
+    # Send Confirmation Email (Async/Background in production, Sync here for safety)
+    from . import email_service
+    video_link = f"https://meet.jit.si/ai-health-{new_appt.id}" # Secure unique link
+    
+    email_service.send_booking_confirmation(
+        to_email=current_user.email or "patient@example.com",
+        patient_name=current_user.full_name or current_user.username,
+        doctor_name=appt.specialist,
+        date_time=dt_str,
+        link=video_link
+    )
+    
     return new_appt
 
 @router.get("/", response_model=list[schemas.AppointmentResponse])
@@ -49,3 +63,19 @@ def get_appointments(
     return db.query(models.Appointment).filter(
         models.Appointment.user_id == current_user.id
     ).order_by(models.Appointment.date_time.asc()).all()
+
+@router.get("/doctors", response_model=list[schemas.DoctorResponse])
+def get_doctors(db: Session = Depends(database.get_db)):
+    """Fetch all users with role='doctor'"""
+    doctors = db.query(models.User).filter(models.User.role == "doctor").all()
+    # Map to DoctorResponse (handling missing profile fields)
+    response = []
+    for doc in doctors:
+        response.append(schemas.DoctorResponse(
+            id=doc.id,
+            full_name=doc.full_name or doc.username,
+            specialization="General Physician", # Hardcoded for now or fetch from profile if stored
+            consultation_fee=doc.consultation_fee or 500.0,
+            profile_picture=doc.profile_picture
+        ))
+    return response

backend/email_service.py

@@ -0,0 +1,46 @@
+"""
+Email Service (Simulated)
+=========================
+Handles sending transactional emails.
+Currently configured for SIMULATION mode to ensure stability.
+Logs emails to server console instead of risking SMTP crashes.
+"""
+import logging
+import os
+
+logger = logging.getLogger(__name__)
+
+def send_booking_confirmation(to_email: str, patient_name: str, doctor_name: str, date_time: str, link: str):
+    """
+    Simulates sending a booking confirmation email.
+    In production, this would use SMTP or SendGrid.
+    """
+    try:
+        # Construct Email Body
+        subject = f"Appointment Confirmed: {doctor_name}"
+        body = f"""
+        Dear {patient_name},
+        
+        Your appointment with {doctor_name} has been confirmed.
+        
+        📅 Date & Time: {date_time}
+        🔗 Video Link: {link}
+        
+        Please join 5 minutes early.
+        
+        Regards,
+        AI Healthcare System
+        """
+        
+        # Log instead of Send (Safety First)
+        logger.info(f"📧 [EMAIL SENT TO {to_email}] Subject: {subject}")
+        logger.info(f"📧 Body: {body.strip()}")
+        
+        # If we had SMTP credentials, we would use them here:
+        # if os.getenv("SMTP_SERVER"):
+        #     send_smtp_email(...)
+            
+        return True
+    except Exception as e:
+        logger.error(f"Failed to send email: {e}")
+        return False

backend/main.py

@@ -46,7 +46,10 @@ def run_migrations():
         "subscription_expiry": "TIMESTAMP", # Changed from DATETIME for Postgres compat
         "razorpay_customer_id": "VARCHAR",
         "created_at": "TIMESTAMP",
-        "role": "VARCHAR"
+        "role": "VARCHAR",
+        "consultation_fee": "FLOAT",
+        # Note: Foreign keys are harder to add via simple script, assuming basic column add for now
+        "doctor_id": "INTEGER" 
     }
 
     try:

backend/models.py

@@ -38,6 +38,9 @@ class User(Base):
     subscription_expiry = Column(DateTime, nullable=True)
     razorpay_customer_id = Column(String, nullable=True)
 
+    # Doctor Specific
+    consultation_fee = Column(Float, default=500.0)
+    
     # AI Memory
     psych_profile = Column(Text, nullable=True) # Long term memory summary
 
@@ -85,7 +88,8 @@ class Appointment(Base):
 
     id = Column(Integer, primary_key=True, index=True)
     user_id = Column(Integer, ForeignKey("users.id"))
-    specialist = Column(String)
+    doctor_id = Column(Integer, ForeignKey("users.id"), nullable=True) # Link to specific doctor
+    specialist = Column(String) # Keep for fallback name display
     date_time = Column(DateTime)
     reason = Column(Text)
     status = Column(String, default="Scheduled") # Scheduled, Completed, Cancelled

backend/schemas.py

@@ -66,20 +66,30 @@ class UserFullResponse(UserResponse):
 
 
 
+
 class AppointmentCreate(BaseModel):
-    specialist: str
+    doctor_id: Optional[int] = None # Link to real doctor
+    specialist: str # Fallback name
     date: str
     time: str
     reason: str
 
 class AppointmentResponse(BaseModel):
     id: int
+    doctor_id: Optional[int] = None
     specialist: str
     date_time: datetime
     reason: str
     status: str
     model_config = ConfigDict(from_attributes=True)
-
+    
+class DoctorResponse(BaseModel):
+    id: int
+    full_name: str
+    specialization: str = "General Physician" # Default for now if not in DB
+    consultation_fee: float
+    profile_picture: Optional[str] = None
+    
 # --- Prediction Schemas ---
 
 class DiabetesInput(BaseModel):

frontend/views/telemedicine_view.py

@@ -19,12 +19,25 @@ def render_telemedicine_page():
     with col1:
         st.markdown("### 📅 Book an Appointment")
         with st.form("booking_form"):
-            specialist = st.selectbox("Select Specialist", [
-                "General Physician (Dr. Smith)",
-                "Cardiologist (Dr. A. Gupta)",
-                "Endocrinologist (Dr. R. Lee)",
-                "Nutritionist (Sarah Jones)"
-            ])
+            # Fetch Doctors Dynamic List
+            from frontend.utils import api
+            doctors = api.fetch_doctors()
+            
+            doctor_map = {}
+            if doctors:
+                for doc in doctors:
+                    label = f"{doc['full_name']} - {doc['specialization']} (₹{doc['consultation_fee']})"
+                    doctor_map[label] = doc
+                    
+                selected_label = st.selectbox("Select Specialist", list(doctor_map.keys()))
+                selected_doc = doctor_map[selected_label]
+            else:
+                st.warning("No doctors are currently available. Using fallback.")
+                selected_doc = None
+                specialist = st.selectbox("Select Specialist", [
+                     "General Physician (Dr. Smith)",
+                     "Cardiologist (Dr. A. Gupta)"
+                ])
 
             d = st.date_input("Preferred Date", min_value=datetime.today())
             t = st.time_input("Preferred Time", value=datetime.now())
@@ -37,14 +50,19 @@ def render_telemedicine_page():
                 if not reason:
                     st.error("Please provide a reason for your visit.")
                 else:
+                    specialist_name = selected_doc['full_name'] if selected_doc else specialist
+                    doctor_id = selected_doc['id'] if selected_doc else None
+                    
                     payload = {
-                        "specialist": specialist,
+                        "doctor_id": doctor_id,
+                        "specialist": specialist_name,
                         "date": str(d),
                         "time": str(t),
                         "reason": reason
                     }
                     if api.book_appointment(payload):
-                        st.success(f"✅ Appointment Confirmed with {specialist}!")
+                        st.success(f"✅ Appointment Confirmed with {specialist_name}!")
+                        st.info("📧 Confirmation email has been sent.")
                         st.balloons()
                         st.rerun()