-
Notifications
You must be signed in to change notification settings - Fork 0
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
[BB-9241] Add learner pathways cert generation #69
base: main
Are you sure you want to change the base?
Conversation
377a160
to
dfd5de3
Compare
853d7d2
to
6a820f9
Compare
dfd5de3
to
971c2af
Compare
@pkulkark, I haven't fully reviewed the PR yet, as I was closing #3. I already rebased your PR after merging it. That said, I have a couple of initial notes:
Also, a small note: we should try to use the term "Learning Paths" instead of "pathways" or "learner pathways" to avoid confusion. |
@Agrendalath Thanks for the insight, that makes a lot of sense.
How about something like
That's a good point.
That makes sense. I'll wait for an update before moving any further here. |
Description
This PR adds the integration with learner pathways so that the cert service can be used to generate certificates for learner pathways. It replaces the
course_id
field withresource_id
andresource_type
fields, so that the cert service can be extended to other types of data resources (not just courses). If a course that is part of the learner pathway offers certificates then the eligibility criteria already defined is used. If not, then it defaults to completion criteria.Supporting Information
OpenCraft Internal Jira ticket: BB-9241
Testing Instructions
retrieve_learner_paths_eligible_users
.custom_options
field:Generate certificate
and verify that all eligible users receive a certificate.