-
Notifications
You must be signed in to change notification settings - Fork 160
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
Recipients list missing issued certificates #560
Comments
We are experiencing the same issue (our own platform + with all our clients). The report 'issued certificates' is empty, although there are certificates issued already. |
I unenrolled a user from the course and refreshed the list of issued certificates and the user was still displayed. However, if you delete the user completely from the site they will not appear. So, this part seems to work as expected. I then enrolled the admin as a student and checked that they were issued a certificate when I ran the cron job and they were not. I then hacked the database to issue the admin a certificate and they were still not listed. So two things to consider -
The problem with making these changes is that people who are used to the behaviour now are going to upgrade and end up with a bunch of people getting issued certificates. I need to think about this. |
I see #376 is what introduced the logic of not sending the certificate to anyone that can manage. |
Hi Mark,
I was confused recently, looking at the recipients list shown in the certificate, and not being able to find some recipients listed in another report. Looking into it, it seems that this is intentional. Certificates issued to users who now have the capability to manage certificates aren't listed, presumably because they are no longer eligible to receive a certificate. I think this is unintuitive behaviour though. I think the recipients list should include everyone who's been issued a certificate, even if they are now no longer eligible to receive one.
The text was updated successfully, but these errors were encountered: