|
Michael Blake made changes - 21/Aug/06 05:01 PM
Hi,
i've attached a new cron_course_pending.php which is called by an new crontab-entry e.g. twice a day. Possible crontab-entry: //0 6,18 * * * root wget -O /var/log/moodle-cron-course-pending http://localhost/moodle/admin/cron-course-pending.php Notice: every time you call this skript admin will receive a email if pending courses are available - so think about your cron-intervall you will call this script... Bertold
Bertold Altaner made changes - 27/Mar/08 05:08 PM
oh. forgot about this, sorry :-O
this should definitely be in 1.9.x
Petr Skoda made changes - 27/Mar/08 05:18 PM
Petr Skoda made changes - 17/Apr/08 05:52 AM
We did something similar, but not in cron.ph instead in course/request.php:
//Begin: Informing Admin notice(get_string('courserequestsuccess')); Thanks to Andre Krüger from Berlin who suggested this to me at moodle.de.
Martin Dougiamas made changes - 15/May/08 03:02 PM
Petr Skoda made changes - 06/Jul/08 12:29 AM
We did something similar to this in course_request.php
but used the form: $admin = get_admin(); This way it is easy to reply to the requesting user and also link to /course/pending.php
Martin Dougiamas made changes - 15/Oct/08 03:00 PM
Tim Hunt made changes - 01/Dec/08 05:56 PM
tjhunt committed 8 files to 'Moodle CVS' - 10/Dec/08 11:57 AM
tjhunt committed 8 files to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 10/Dec/08 11:57 AM
Tim Hunt made changes - 10/Dec/08 11:58 AM
martignoni committed 2 files to 'Lang CVS' - 10/Dec/08 06:27 PM
Tim and Petr, thanks a lot for fixing this issue
It's now noted in the documentation: http://docs.moodle.org/en/Course_request Thinking a bit more about this feature, it seems weird to enter a username for the user to be notified when new course requested.
Why not instead have email notification sent to all admins or have a field for entering an email address to which notification should be sent?
Helen Foster made changes - 17/Dec/08 08:03 AM
Hi Tim,
In Administration > Security > Notifications the notifyloginfailures setting has 3 options - Nobody, Administrator and All administrators (I'm not sure how this is working since the implementation of roles!) Alternatively, perhaps we could have a notifycourserequest capability similar to mod/quiz:emailnotifysubmission. The attached patch (HEAD only) changes to admin setting to be a multi-select box with a list of all the users who have the 'moodle/site:approvecourse' capability, and lets you select any number of those users.
What do people think? And should we do something similar for notify login failures?
Tim Hunt made changes - 06/Jan/09 03:18 PM
I think the multi-select box for course notification is great! It's really clever that only relevant users are listed.
A Nobody option would be helpful for people like me who don't know about CTRL+click. Alternatively, the following text could be added to the lang string: Multiple users may be selected/deselected by holding down the Apple or Ctrl key whilst clicking on the users' names. +1 for a multi-select box for notifyloginfailures. I can't find any other admin settings which would benefit from the same treatment.
tjhunt committed 5 files to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 08/Jan/09 04:05 PM
tjhunt committed 8 files to 'Moodle CVS' - 08/Jan/09 04:07 PM
Tim Hunt made changes - 08/Jan/09 04:07 PM
martignoni committed 1 file to 'Lang CVS' - 11/Jan/09 06:10 PM
Mitsuhiro Yoshida committed 3 files to 'Lang CVS' - 12/Jan/09 03:42 PM
I think there's a problem here... please see:
Howard Miller made changes - 13/Jan/09 08:09 PM
Tim Hunt made changes - 14/Jan/09 01:12 PM
tjhunt committed 1 file to 'Moodle CVS' on branch 'MOODLE_19_STABLE' - 16/Jan/09 05:34 PM
Tim, thanks a lot for further improving course request notifications. Everything seems to be working fine
Helen Foster made changes - 21/Jan/09 04:44 PM
notifyloginfailures improvement noted in the documentation:
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
mtrace("Checking course creation requests");
if ($pending = get_records("course_request")) { $admin = get_admin(); $subject = "Course requests pending approval"; $messagetext = "Course requests have been submitted for approval. Please check pending.html."; email_to_user($admin,$admin,$subject,$messagetext); }
mtrace("Course requests checked");