-
Notifications
You must be signed in to change notification settings - Fork 481
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
Add new self-paced PL certificate image and generation #56291
Add new self-paced PL certificate image and generation #56291
Conversation
…cates/virtual-pl-certificate
…cates/virtual-pl-certificate
…cates/virtual-pl-certificate
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Just one tiny nit that's not worth a rebuild if no other feedback
dashboard/lib/certificate_image.rb
Outdated
end | ||
|
||
def self.prefilled_title_course?(course) | ||
certificate_template_for(course) != 'blank_certificate.png' | ||
certificate_template_for(course) != 'blank_certificate.png' && certificate_template_for(course) != 'self_paced_pl_certificate.png' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: ['blank_certificate.png', 'self_paced_pl_certificate.png'].include? certificate_template_for(course)
(not worth a rebuild though if no other changes)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like that! Don't mind a rebuild -- this PR isn't urgent :)
Adds a certificate for self-paced PL! This certificate isn't linked to anywhere yet, but if you happen to know the obfuscated url for the course, this would allow you to generate a PL-specific certificate.
Examples:
Links
product spec: https://docs.google.com/document/d/1Ee02kw_ei2x6wOS06K768dC-kUhGDlydjoE6BZipbPE/edit
Testing story
Some unit testings, but largely manual testing