Add course_link field to certificate model and implement delete functionality

Co-authored-by: Copilot <copilot@github.com>

Author: orabe

backend/app/crud.py

@@ -29,6 +29,7 @@ def create_certificate(db: Session, cert_in: schemas.CertificateCreate):
         course_level=cert_in.course_level,
         course_format=cert_in.course_format,
         instruction_language=cert_in.instruction_language,
+        course_link=cert_in.course_link,
         issuer=cert_in.issuer or "MathCodeLab",
         instructor=cert_in.instructor or "Mohammad Orabe",
     )
@@ -44,3 +45,13 @@ def revoke_certificate(db: Session, certificate_id: str, reason: str = None):
         return None
     # Revocation removed: function retained only for compatibility but does nothing
     return None
+
+
+def delete_certificate(db: Session, certificate_id: str):
+    cert = get_certificate_by_public_id(db, certificate_id)
+    if not cert:
+        return None
+
+    db.delete(cert)
+    db.commit()
+    return cert

backend/app/database.py

@@ -53,6 +53,8 @@ def ensure_certificate_schema(bind=None):
         missing_statements.append("ALTER TABLE certificates ADD COLUMN course_format VARCHAR")
     if "instruction_language" not in existing_columns:
         missing_statements.append("ALTER TABLE certificates ADD COLUMN instruction_language VARCHAR")
+    if "course_link" not in existing_columns:
+        missing_statements.append("ALTER TABLE certificates ADD COLUMN course_link VARCHAR")
 
     if not missing_statements:
         return

backend/app/main.py

@@ -70,6 +70,7 @@ def verify_certificate(certificate_id: str, db: Session = Depends(get_db)):
         "certificate_id": cert.certificate_id,
         "student_name": cert.student_name,
         "course_title": cert.course_title,
+        "course_link": cert.course_link,
         "completion_date": cert.completion_date,
         "duration_hours": cert.duration_hours,
         "issuer": cert.issuer,
@@ -94,4 +95,20 @@ def list_certificates(
     api_key: str = Depends(security.verify_api_key)
 ):
     certs = db.query(models.Certificate).all()
-    return certs
+    return certs
+
+
+@app.delete("/admin/certificates/{certificate_id}", response_model=schemas.CertificateDeleteResponse)
+def delete_certificate(
+    certificate_id: str,
+    db: Session = Depends(get_db),
+    api_key: str = Depends(security.verify_api_key)
+):
+    cert = crud.delete_certificate(db, certificate_id)
+    if not cert:
+        raise HTTPException(status_code=404, detail="Certificate not found")
+
+    return {
+        "message": "Certificate deleted",
+        "certificate_id": certificate_id,
+    }

backend/app/models.py

@@ -15,6 +15,7 @@ class Certificate(Base):
     course_level = Column(String, nullable=True)
     course_format = Column(String, nullable=True)
     instruction_language = Column(String, nullable=True)
+    course_link = Column(String, nullable=True)
 
     issuer = Column(String, default="MathCodeLab", nullable=False)
     instructor = Column(String, default="Mohammad Orabe", nullable=False)

backend/app/schemas.py

@@ -11,6 +11,7 @@ class CertificateBase(BaseModel):
     course_level: Optional[str] = None
     course_format: Optional[str] = None
     instruction_language: Optional[str] = None
+    course_link: Optional[str] = None
     issuer: Optional[str] = "MathCodeLab"
     instructor: Optional[str] = "Mohammad Orabe"
 
@@ -21,13 +22,15 @@ class CertificateOut(CertificateBase):
     certificate_id: str
     class Config:
         from_attributes = True
+    course_link: Optional[str] = None
 
 class CertificateVerificationResponse(BaseModel):
     status: str
     certificate_id: str
     verification_url: Optional[str] = None
     student_name: Optional[str] = None
     course_title: Optional[str] = None
+    course_link: Optional[str] = None
     completion_date: Optional[str] = None
     duration_hours: Optional[int] = None
     issuer: Optional[str] = None
@@ -39,5 +42,10 @@ class CertificateVerificationResponse(BaseModel):
     course_level: Optional[str] = None
     course_format: Optional[str] = None
     instruction_language: Optional[str] = None
+
+
+class CertificateDeleteResponse(BaseModel):
+    message: str
+    certificate_id: str
 
 # Revocation support removed: no revoke schema

backend/scripts/create_certificate.py

@@ -48,6 +48,7 @@ def main():
     course_level = input("Course level (e.g. Master-level, Bachelor-level, High School-level): ")
     course_format = input("Course format (e.g. Online (via Zoom), In-person, Hybrid): ")
     instruction_language = input("Instruction language (e.g. English, German, Arabic): ") 
+    course_link = input("Course link (optional, e.g. https://...): ").strip() or None
     cert_in = schemas.CertificateCreate(
         student_name=student_name,
         course_title=course_title,
@@ -58,6 +59,8 @@ def main():
         course_level=course_level,
         course_format=course_format,
         instruction_language=instruction_language
+        ,
+        course_link=course_link
     )
     cert = crud.create_certificate(db, cert_in)
     print(f"Created certificate: {cert.certificate_id}")

backend/scripts/view_database.py

@@ -0,0 +1,54 @@
+import os
+import sys
+
+from dotenv import load_dotenv
+from sqlalchemy.orm import Session
+
+sys.path.append(os.path.dirname(os.path.dirname(__file__)))
+
+from app.database import SessionLocal
+from app.models import Certificate
+
+
+load_dotenv()
+
+
+def view_database(limit: int = 50):
+	db: Session = SessionLocal()
+	try:
+		certificates = (
+			db.query(Certificate)
+			.order_by(Certificate.created_at.desc())
+			.limit(limit)
+			.all()
+		)
+
+		if not certificates:
+			print("No certificates found in the database.")
+			return
+
+		print(f"\nShowing up to {limit} certificates:\n")
+
+		for cert in certificates:
+			print("=" * 50)
+			print(f"Certificate ID : {cert.certificate_id}")
+			print(f"Student Name   : {cert.student_name}")
+			print(f"Course Title   : {cert.course_title}")
+			print(f"Completion Date: {cert.completion_date}")
+			print(f"Duration       : {cert.duration_hours} hours")
+			print(f"Issuer         : {cert.issuer}")
+			print(f"Instructor     : {cert.instructor}")
+			print(f"Course Link    : {cert.course_link or '-'}")
+			print(f"Status         : {getattr(cert, 'status', 'valid')}")
+			print(f"Created At     : {cert.created_at}")
+			print("=" * 50)
+
+	except Exception as e:
+		print("Error while fetching data:")
+		print(e)
+	finally:
+		db.close()
+
+
+if __name__ == "__main__":
+	view_database()