Update backend/api/email_utils.py

backend/api/email_utils.py

@@ -9,17 +9,13 @@ from django.conf import settings
 # --- CONFIGURATION ---
 # 1. HARDCODE YOUR KEY HERE FOR TESTING (Remove before deploying to GitHub)
 SENDGRID_API_KEY = os.environ.get('SENDGRID_API_KEY') 
-SENDER_EMAIL = "gamingyash54@gmail.com"  # <--- MUST match the Single Sender you verified
+SENDER_EMAIL = "gamingyash54@gmail.com" 
 # ---------------------
 
 def send_html_email(subject, recipient_list, html_content, pdf_buffer=None, filename="Report.pdf"):
     """
-    Sends an email using SendGrid API.
+    Sends an email using SendGrid API (Bypasses Gmail SMTP).
     """
-    if not SENDGRID_API_KEY:
-        print("❌ CRITICAL ERROR: SENDGRID_API_KEY is missing! Check your .env file.")
-        return
-
     message = Mail(
         from_email=SENDER_EMAIL,
         to_emails=recipient_list,
@@ -38,13 +34,14 @@ def send_html_email(subject, recipient_list, html_content, pdf_buffer=None, file
         message.attachment = attachment
 
     try:
-        print(f"🚀 Attempting to send email to {recipient_list}...")
+        print(f"🚀 Sending email via SendGrid to {recipient_list}...")
         sg = SendGridAPIClient(SENDGRID_API_KEY)
         response = sg.send(message)
-        print(f"✅ Email Sent! Status Code: {response.status_code}")
+        print(f"✅ SendGrid Status: {response.status_code}")
     except Exception as e:
-        print(f"❌ EMAIL FAILED: {str(e)}")
-        # We print the error but do not raise it, to prevent the app from crashing.
+        print(f"❌ SendGrid Failed: {str(e)}")
+        # Don't raise here to prevent crashing the view if email fails
+        pass
 
 def get_medical_email_template(patient_name, test_date, risk_level, confidence):
     """
@@ -67,11 +64,14 @@ def get_medical_email_template(patient_name, test_date, risk_level, confidence):
         <div style="padding: 30px;">
             <h3>Hello {patient_name},</h3>
             <p>Your analysis from {test_date} is complete.</p>
+            
             <div style="background: #f8fafc; padding: 15px; border-radius: 6px; margin: 20px 0;">
                 <p><strong>Result:</strong> <span style="color: {color}; font-weight: bold;">{icon} {risk_level} Risk</span></p>
                 <p><strong>AI Confidence:</strong> {confidence}%</p>
             </div>
+            
             <p>Please find the detailed PDF report attached.</p>
+            
             <div style="text-align: center; margin-top: 30px;">
                 <a href="{dashboard_link}" style="background-color: #2563eb; color: white; padding: 12px 24px; text-decoration: none; border-radius: 6px; font-weight: bold;">Login to Dashboard</a>
             </div>
@@ -86,50 +86,35 @@ def send_appointment_status_email(recipient_email, patient_name, doctor_name, ap
     subject = f"Appointment Update - Dr. {doctor_name}"
     
     if status == 'confirmed':
-        color = "#059669"  # Green
+        color = "#059669" # Green
         status_text = "CONFIRMED"
-        message_body = f"Your appointment with Dr. {doctor_name} has been <strong>confirmed</strong> for {appointment_date}."
-        if doctor_note:
-            message_body += f"<br><br><strong>Doctor's Note:</strong> <div style='background:#f0fdf4; border-left: 4px solid #059669; padding:10px; margin-top:5px;'><em>{doctor_note}</em></div>"
-
-    elif status == 'cancelled':
-        color = "#dc2626"  # Red
-        status_text = "CANCELLED"
-        message_body = f"We're sorry, your appointment with Dr. {doctor_name} scheduled for {appointment_date} has been <strong>cancelled</strong>."
-        if doctor_note:
-            message_body += f"<br><br><strong>Reason:</strong> <div style='background:#fef2f2; border-left: 4px solid #dc2626; padding:10px; margin-top:5px;'><em>{doctor_note}</em></div>"
-        message_body += "<br><br>Please book a new appointment at your convenience."
-
-    elif status == 'completed':
-        color = "#2563eb"  # Blue
-        status_text = "COMPLETED"
-        message_body = f"Your appointment with Dr. {doctor_name} on {appointment_date} has been marked as <strong>completed</strong>. Thank you for using RespireX."
-
+        message_body = f"Your appointment has been <strong>confirmed</strong> for {appointment_date}."
     else:
-        # pending or time/note update (no status change)
-        color = "#ca8a04"  # Yellow/Orange
-        status_text = "UPDATE / RESCHEDULE"
-        message_body = f"Dr. {doctor_name} has sent an update regarding your appointment."
-        message_body += f"<br><br><strong>New Proposed Time:</strong> {appointment_date}"
+        color = "#e11d48" # Red
+        status_text = "NEEDS RESCHEDULING"
+        message_body = f"Dr. {doctor_name} has sent a message regarding your appointment request for {appointment_date}."
         if doctor_note:
-            message_body += f"<br><br><strong>Doctor's Message:</strong> <div style='background:#fffbeb; border-left: 4px solid #ca8a04; padding:10px; margin-top:5px;'><em>{doctor_note}</em></div>"
+            message_body += f"<br><br><strong>Doctor's Note:</strong> <em style='background:#fff1f2; padding:5px;'>{doctor_note}</em>"
 
     html_content = f"""
     <div style="font-family: Arial, sans-serif; max-width: 600px; margin: 0 auto; border: 1px solid #eee; border-radius: 8px;">
         <div style="background-color: #0f172a; color: white; padding: 20px; text-align: center;">
-            <h2 style="margin:0;">Appointment Status</h2>
+            <h2 style="margin:0;">Appointment Update</h2>
         </div>
         <div style="padding: 30px;">
             <h3>Hello {patient_name},</h3>
             
-            <div style="background: #f8fafc; padding: 15px; border-radius: 6px; margin: 20px 0; border: 1px solid {color};">
+            <div style="background: #f8fafc; padding: 15px; border-radius: 6px; margin: 20px 0; border-left: 4px solid {color};">
                 <p style="margin:0 0 10px 0; color: {color}; font-weight: bold; font-size: 1.1em;">STATUS: {status_text}</p>
-                <div style="margin:0; line-height: 1.5;">{message_body}</div>
+                <p style="margin:0; line-height: 1.5;">{message_body}</p>
             </div>
             
-            <p style="color: #64748b; font-size: 0.9em;">Please check your dashboard for more details.</p>
+            <p style="color: #64748b; font-size: 0.9em;">Please log in to your dashboard to view more details or book a new slot.</p>
         </div>
     </div>
     """
     
-    send_html_email(subject, [recipient_email], html_content)
+    try:
+        send_html_email(subject, [recipient_email], html_content)
+    except Exception as e:
+        print(f"Failed to send appointment email: {e}")