forked from markward/enrol_autoenrol
-
Notifications
You must be signed in to change notification settings - Fork 3
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
When set, it will send an email to the user enrolled. Most of this code is copied and slightly modified from enrol_self, but without the default message. Fixes MDLSITE-3926 and markward#10
- Loading branch information
1 parent
9bc16b7
commit f6f4b3c
Showing
4 changed files
with
88 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
f6f4b3c
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.
Dan,
I see some back and forth with David about using this and your addition of pre-3.1 messaging. Are you still using this plugin? We have been using it with 3.0 and planning to upgrade to 3.1, but noticing an error message that seems to indicate some changes in how 3.1 expects messages to be implemented.
If you're not using it - has something else superseded it?
f6f4b3c
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 think that this is still in use on learn.moodle.net (perhaps with some more fixes). @mudrd8mz might know more. We probably should find some time to send a formal pull request
f6f4b3c
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.
We use rev 3b07f70 of this plugin with some recent fix added by Karen Holland that might be related to the problem mentioned by Jeremy. I asked Karen to check this discussion.
f6f4b3c
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 appreciate it. If there's a pull request I can at least use that to grab the code, even if the original developer seems to have stopped working on the plugin. It is very useful - especially for corporate environments where we don't have an SIS.