Upload 2 files

app.py

@@ -264,8 +264,10 @@ def preview_with_test_name(template_file, test_name, placeholder_text="[NAME]"):
         return f"❌ Error: {str(e)}"
 
 def send_email_with_certificate(recipient_email, recipient_name, cert_file_path):
-    """Send certificate via email as PDF"""
+    """Send certificate via email as PDF or DOCX"""
     try:
+        print(f"📧 Preparing email for {recipient_name} ({recipient_email})...")
+        
         # Create message
         msg = MIMEMultipart()
         msg['From'] = email_config['sender_email']
@@ -279,7 +281,11 @@ def send_email_with_certificate(recipient_email, recipient_name, cert_file_path)
         
         # Check if file exists and is a valid PDF or DOCX
         if not os.path.exists(cert_file_path):
-            return False, f"Certificate file not found: {cert_file_path}"
+            error_msg = f"Certificate file not found: {cert_file_path}"
+            print(f"❌ {error_msg}")
+            return False, error_msg
+        
+        print(f"📎 Attaching: {os.path.basename(cert_file_path)}")
         
         # Determine file type and set appropriate MIME type
         file_ext = os.path.splitext(cert_file_path)[1].lower()
@@ -303,15 +309,20 @@ def send_email_with_certificate(recipient_email, recipient_name, cert_file_path)
             part.add_header('Content-Type', mime_type)
             msg.attach(part)
         
+        print(f"🚀 Sending email to {recipient_email}...")
+        
         # Send email
         with smtplib.SMTP(email_config['smtp_server'], email_config['smtp_port']) as server:
             server.starttls()
             server.login(email_config['sender_email'], email_config['sender_password'])
             server.send_message(msg)
         
+        print(f"✅ Email sent successfully to {recipient_name}")
         return True, None
     except Exception as e:
-        return False, str(e)
+        error_msg = f"Email error: {str(e)}"
+        print(f"❌ {error_msg}")
+        return False, error_msg
 
 def send_email_with_retry(recipient_email, recipient_name, cert_file_path, max_retries=3):
     """Send email with retry logic and delay"""
@@ -355,10 +366,23 @@ def generate_single_certificate(template_path, name, placeholder_text):
 def process_certificates(template_file, csv_file, send_emails, placeholder_text, progress=gr.Progress()):
     """Main function to process all certificates with threading"""
     try:
+        print("\n" + "="*60)
+        print("🚀 Starting certificate processing...")
+        print(f"📋 Send emails: {send_emails}")
+        print(f"🔤 Placeholder text: '{placeholder_text}'")
+        print("="*60 + "\n")
+        
         # Validate email config if sending emails
         if send_emails:
             if not all([email_config.get('smtp_server'), email_config.get('sender_email'), email_config.get('sender_password')]):
-                return "❌ Please configure email server settings first!", None
+                error_msg = "❌ Please configure email server settings first!"
+                print(error_msg)
+                return error_msg, None
+            
+            print(f"📧 Email configured:")
+            print(f"  Server: {email_config['smtp_server']}:{email_config['smtp_port']}")
+            print(f"  Sender: {email_config['sender_email']}")
+            print(f"  Subject: {email_config.get('email_subject', 'Your Certificate')}")
         
         # Update placeholder text
         setup_text_replacement(placeholder_text)
@@ -366,17 +390,27 @@ def process_certificates(template_file, csv_file, send_emails, placeholder_text,
         # Load template
         template_path = load_template(template_file)
         if template_path is None:
-            return "❌ Could not load template file. Please upload a valid Word document (.docx).", None
+            error_msg = "❌ Could not load template file. Please upload a valid Word document (.docx)."
+            print(error_msg)
+            return error_msg, None
+        
+        print(f"✅ Template loaded: {os.path.basename(template_path)}\n")
         
         # Read CSV
         df = pd.read_csv(csv_file)
+        print(f"📊 CSV loaded: {len(df)} students found")
+        print(f"   Columns: {', '.join(df.columns.tolist())}\n")
         
         # Validate CSV columns
         if 'Name' not in df.columns:
-            return "❌ CSV must have a 'Name' column!", None
+            error_msg = "❌ CSV must have a 'Name' column!"
+            print(error_msg)
+            return error_msg, None
         
         if send_emails and 'Email address' not in df.columns:
-            return "❌ CSV must have an 'Email address' column for sending emails!", None
+            error_msg = "❌ CSV must have an 'Email address' column for sending emails!"
+            print(error_msg)
+            return error_msg, None
         
         results = []
         total = len(df)
@@ -384,6 +418,8 @@ def process_certificates(template_file, csv_file, send_emails, placeholder_text,
         failed = 0
         generated_files = []
         
+        print(f"📄 Phase 1: Generating {total} certificates in parallel (max {MAX_WORKERS} workers)...\n")
+        
         # Phase 1: Generate all certificates in parallel with progress
         cert_futures = {}
         
@@ -392,9 +428,12 @@ def process_certificates(template_file, csv_file, send_emails, placeholder_text,
             for idx, row in df.iterrows():
                 name = row['Name']
                 email = row.get('Email address', '')
+                print(f"  📝 Queued: {name}")
                 future = executor.submit(generate_single_certificate, template_path, name, placeholder_text)
                 cert_futures[future] = (name, email)
             
+            print(f"\n⚙️ Processing {len(cert_futures)} certificates...\n")
+            
             # Process completed futures with progress
             completed_count = 0
             for future in as_completed(cert_futures):
@@ -405,29 +444,38 @@ def process_certificates(template_file, csv_file, send_emails, placeholder_text,
                     cert_file, error = future.result()
                     
                     if cert_file and os.path.exists(cert_file):
+                        file_ext = os.path.splitext(cert_file)[1]
+                        print(f"  ✅ [{completed_count}/{total}] Generated {file_ext} for: {name}")
                         generated_files.append((cert_file, name, email))
                         if not send_emails:
                             successful += 1
                             results.append(f"✅ {name}")
                     else:
+                        print(f"  ❌ [{completed_count}/{total}] Failed for: {name} - {error if error else 'Unknown error'}")
                         failed += 1
                         results.append(f"❌ {name}: {error if error else 'Generation failed'}")
                         
                 except Exception as e:
+                    print(f"  ❌ [{completed_count}/{total}] Exception for: {name} - {str(e)}")
                     failed += 1
                     results.append(f"❌ {name}: {str(e)}")
                 
                 # Update progress
                 progress(completed_count / total, desc=f"📄 Generated {completed_count}/{total} certificates")
         
+        print(f"\n✅ Phase 1 complete: {len(generated_files)} certificates generated\n")
+        
         # Phase 2: Send emails sequentially with rate limiting and progress
         if send_emails and generated_files:
             email_successful = 0
             email_failed = 0
             email_total = len(generated_files)
             
+            print(f"📧 Phase 2: Sending {email_total} emails sequentially...\n")
+            
             for idx, (cert_file, name, email) in enumerate(generated_files):
                 try:
+                    print(f"  📧 [{idx+1}/{email_total}] Sending to: {name} ({email})...")
                     success, error = send_email_with_retry(email, name, cert_file)
                     
                     if success:
@@ -438,6 +486,7 @@ def process_certificates(template_file, csv_file, send_emails, placeholder_text,
                                 results[i] = f"✅ {name} ({email})"
                                 break
                     else:
+                        print(f"    ⚠️ Email failed: {error}")
                         email_failed += 1
                         # Update existing result
                         for i in range(len(results)):
@@ -449,6 +498,7 @@ def process_certificates(template_file, csv_file, send_emails, placeholder_text,
                     progress((idx + 1) / email_total, desc=f"📧 Sent {idx + 1}/{email_total} emails")
                     
                 except Exception as e:
+                    print(f"    ⚠️ Email exception: {str(e)}")
                     email_failed += 1
                     for i in range(len(results)):
                         if name in results[i]:
@@ -457,6 +507,8 @@ def process_certificates(template_file, csv_file, send_emails, placeholder_text,
             
             successful = email_successful
             failed = email_failed
+            
+            print(f"\n✅ Phase 2 complete: {email_successful} emails sent, {email_failed} failed\n")
         
         # Create ZIP file with all certificates
         zip_path = None
@@ -464,6 +516,7 @@ def process_certificates(template_file, csv_file, send_emails, placeholder_text,
         
         if all_cert_files:
             try:
+                print(f"📦 Creating ZIP file with {len(all_cert_files)} certificates...")
                 # Create a temporary directory for the ZIP
                 zip_dir = tempfile.mkdtemp()
                 timestamp = datetime.now().strftime("%Y%m%d_%H%M%S")
@@ -477,19 +530,28 @@ def process_certificates(template_file, csv_file, send_emails, placeholder_text,
                             # Add file to ZIP with just the filename (not full path)
                             zipf.write(cert_file, os.path.basename(cert_file))
                 
-                print(f"ZIP file created: {zip_path}")
+                print(f"✅ ZIP file created: {zip_filename} ({os.path.getsize(zip_path) / 1024:.1f} KB)\n")
             except Exception as e:
-                print(f"Error creating ZIP: {e}")
+                print(f"❌ Error creating ZIP: {e}\n")
                 zip_path = None
         
         # Create summary
+        print("="*60)
+        print(f"🎉 Processing complete!")
+        print(f"  Total: {total} | Success: {successful} | Failed: {failed}")
+        print("="*60 + "\n")
+        
         summary = f"Processed: {total} | Success: {successful} | Failed: {failed}\n\n"
         summary += "\n".join(results)
         
         return summary, zip_path
         
     except Exception as e:
-        return f"❌ Error: {str(e)}", None
+        error_msg = f"❌ Error: {str(e)}"
+        print(error_msg)
+        import traceback
+        print(traceback.format_exc())
+        return error_msg, None
 
 # Create Gradio Interface
 with gr.Blocks(title="Certificate Generator & Email Sender", theme=gr.themes.Soft()) as app: