Skip to content

Conversation

@brianhogg
Copy link
Contributor

@brianhogg brianhogg commented Aug 27, 2025

Description

Adds new Email notification for Certificate Earned, and new Engagement for "student earned a certificate"

Also allows the "related post ID" to be optional, so an engagement can apply to any Certificate, Course, etc.

TODO:

  • Improved link that lets them login if they're not (and sharing isn't enabled) and view the certificate immediately, vs. the current 404 error. Will be handled by Redirect Certificate Links to Login if Private #3050
  • Add certificate link to the default notification email template (should be there with the footer, but email notification not showing it?)
  • Add merge code for certificate link for engagement

How has this been tested?

Manually

Screenshots

Checklist:

  • This PR requires and contains at least one changelog file.
  • My code has been tested.
  • My code passes all existing automated tests.
  • My code follows the LifterLMS Coding & Documentation Standards.

@ideadude ideadude moved this to Awaiting Review in Development Aug 27, 2025
@brianhogg brianhogg moved this from Awaiting Review to To do in Development Aug 27, 2025
@brianhogg brianhogg marked this pull request as ready for review November 18, 2025 14:00
@brianhogg brianhogg moved this from To do to Review in Progress in Development Nov 18, 2025
@brianhogg brianhogg added this to the 10.0 milestone Nov 18, 2025
@brianhogg brianhogg moved this from Review in Progress to Awaiting Review in Development Nov 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Awaiting Review

Development

Successfully merging this pull request may close these issues.

2 participants