Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Attendance completion certificate #1477

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

Xpirix
Copy link
Contributor

@Xpirix Xpirix commented Jan 21, 2025

Fix for #1455

Feature changes

  • Added a new column Certificate Type in the Certificate model and showed it in the course attendees' list.
  • The default type of certificate shown in the course attendees' list is the course's certificate type. This is useful for existing courses that have already issued certificates.
  • Pushing the Issue Certificate using credits will open a modal to choose the Certificate type of the attendee
  • The chosen certificate type will show on the create certificate confirmation page
  • Regenerating one or all certificate(s) will not change the certificate type
  • To change a certificate type, the existing certificate needs to be revoked first.
  • Updated the unit tests
attendee-certificate-type.mov

@Xpirix Xpirix requested a review from dimasciput January 22, 2025 06:39
@Xpirix Xpirix marked this pull request as ready for review January 22, 2025 06:40
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.

1 participant