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

Send issued certificates to teacher's email too #130

Open
bozoh opened this issue Jul 10, 2015 · 5 comments
Open

Send issued certificates to teacher's email too #130

bozoh opened this issue Jul 10, 2015 · 5 comments
Milestone

Comments

@bozoh
Copy link
Owner

bozoh commented Jul 10, 2015

Hi Carlos,

In my case, my teachers need to upload learner's certificates to an internal system, Is it possible to have both the student AND the teacher emailed the certificate when the student finishes a course?

I know that the teacher can log into moodle to find the certificate, but it would be great if they could have the certificate delivered to their inbox...

Thanks,

Peter.

@bozoh bozoh added this to the 2.3.0 milestone Jul 10, 2015
@bozoh
Copy link
Owner Author

bozoh commented Jul 10, 2015

@lachlan-00
Copy link

please add this function. my teachers would prefer a copy of the pdf for each student rather than a link to the bulk list.

Would it be possible to just use get_teachers() in the student email function to send to them as well?

@lachlan-00
Copy link

just something like this after line 1191 in liblocal.php inside the send_certificade_email function:

        $ret = email_to_user($user, $from, $subject, $message, $messagehtml, $relativefilepath, $file->get_filename());
+        //get teacher emails
+        foreach ($this->get_teachers() as $teacher) {
+            $tmpmail = $teacher->user->email;
+            @email_to_user($tmpmail, $from, $subject, $message, $messagehtml, $relativefilepath, $file->get_filename());
+        }
        @unlink($fullfilepath);

@lachlan-00
Copy link

https://github.com/lachlan-00/moodle-mod_simplecertificate/commit/6a0539723e966ade2d4c304dc2fce068eae19a9b

i'll look at adding an option to enable/disable this as well

@lachlan-00
Copy link

#133

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants