Hi,

At the moment, in order to send a push notification a user must be granted the permission 'administer site configuration' which is a high level admin permission. I think it would be beneficial to create a new permission to allow a lower grade of admin to send the push notifications. I propose a permission of 'send push notifications'

Kind regards,
John

CommentFileSizeAuthor
#1 push_notifications-new_send_notifications_permission-1765196.patch2.24 KBjohnennew

Comments

johnennew’s picture

johnennew commented
StatusFileSize
new push_notifications-new_send_notifications_permission-1765196.patch2.24 KB

Hi,

Here is a patch suggestion for the finer permission control on the functionality of this module.

Kind regards,
john

johnennew’s picture

johnennew commented
Status: Active » Needs review

changed status to needs review

haagendazs’s picture

haagendazs commented
Assigned: Unassigned » haagendazs
Status: Needs review » Closed (fixed)

Hey ceng: Awesome patch, thanks for sharing. I reviewed this and committed the changes to the 7.x-1.x branch: http://drupalcode.org/project/push_notifications.git/commit/eb11bc24d2c7...

johnennew’s picture

johnennew commented

cool, thanks!

haagendazs’s picture

haagendazs commented
Status: Closed (fixed) » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.