Would it be possible to add a permission to allow a user to administer the list of pre-approved e-mails /domains?

CommentFileSizeAuthor
#4 admin-permissions-1243194-04.patch2.57 KBbenshell
#1 register_preapproved.module.patch1.97 KBMac Clemmens

Comments

Mac Clemmens’s picture

Mac Clemmens commented
Status: Active » Needs review
StatusFileSize
new register_preapproved.module.patch1.97 KB

Here's a quick patch to try.

johnhanley’s picture

johnhanley commented

It's a reasonable suggestion, but your patch won't work because you didn't correctly prefix hook_perm.

I also prefer the perm name "administer pre-approved patterns", which applies to both email addresses and domain names.

Mac Clemmens’s picture

Mac Clemmens commented

OK -- I definitely created a patch too quickly! :) I completely agree with your suggestion. @benshell will create a git-friendly patch and post it here.

benshell’s picture

benshell commented
StatusFileSize
new admin-permissions-1243194-04.patch2.57 KB

Here is an updated patch.

johnhanley’s picture

johnhanley commented
Status: Needs review » Closed (fixed)

Added to release 6.x-1.2. Thanks!