Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 16, 2025

Plan: Update Certificate Entity to Comply with Certificate Creation Rules

  • Analyze current Certificate entity and identify missing fields per .github/instructions/certifcate.instructions.md
  • Update Certificate entity to include all required fields:
    • Add SerialNumber (unique, sequential, human-readable)
    • Add HolderName (derived from Student name, validated, sanitized)
    • Add CourseTitle (from Course name)
    • Rename IssueDate to IssueDateUtc (ensure UTC)
    • Add ExpiryDateUtc (nullable)
    • Add IssuedBy (authority identifier)
    • Add VerificationUrl (HTTPS endpoint)
    • Rename/update DigitalSignature to SignatureHash (SHA-256)
    • Add Version (semantic version)
    • Add Status (Active | Revoked | Expired)
  • Add revocation fields (RevocationReason, RevokedDateUtc)
  • Update validation attributes to match rules
  • Update CertificateDto to match new entity structure
  • Update CertificateService to implement generation workflow
  • Add certificate verification endpoint controller
  • Run tests to ensure no breaking changes
  • Run CodeQL security checks
  • Request code review
Original prompt

This section details on the original issue you should resolve

<issue_title>Draft issue placeholder</issue_title>
<issue_description>This is a draft issue created to capture a new request. Please provide additional details so the issue can be completed.</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

Copilot AI requested a review from Hemavathi15sg December 16, 2025 05:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Draft issue placeholder

1 participant