This seems very cumbersome.

would the api key for the site be enough?

If so I want to remove it from the user area to reduce confusion.

Thanks,

CommentFileSizeAuthor
#6 specifyRole-655648-6.patch1.38 KBwjackson

Comments

Archnemysis’s picture

Category: bug » support

If you want each user to be able to have custom notifications sent to their iPhone, than each user will need to have their own API key.

If you only want to use it to send system notifications to a site admin for example, than you only need the one key for the site, and only that user would receive notifications.

acrollet’s picture

Status: Active » Closed (works as designed)

Hi there,

if you want to remove the per-user API key form, simply disable the Prowl User Integration module.

cheers,

Adrian

thedavidmeister’s picture

Status: Closed (works as designed) » Active

Maybe i misunderstand what's going on, but it seems a little inflexible.

I have a site for an online mag.

There's Editors and Contributors.

When a Contributor posts, i'd like the same message pushed to all Editors via prowl.

I really *don't* want the prowl API key to appear when a new user registers, or on the user form for anyone except editors.

I'd like to use the Prowl Messaging module to let the Messaging framework handle the settings for each user.

So, I tried to follow the advice here and disable the Prowl User Integration module, but the Prowl Messaging module has that listed as a dependency.

What's the best plan of attack?

acrollet’s picture

Category: support » feature

Hi there,

sorry it took a while to respond - looks like basically you would like to restrict prowl API keys to a certain role(s). This is a very reasonable request, but not one I have time for at the moment. I will certainly be happy to review a patch, you'll want to look at the implementation of hook_user() in the prowl_user.module file.

thedavidmeister’s picture

thanks for looking at this, sorry but i'm not using this module any more. We've come up with an alternative solution for our use case.

wjackson’s picture

Status: Active » Needs review
StatusFileSize
new1.38 KB

I too needed to be able to specify which roles could access Prowl, and this patch does the trick. I was able to tackle the issue by adding an additional permission of "View Prowl Settings", and by toggling which role can view the settings; you can effectively hide all functionality from any role that you don't specify. This was a quick way to get around the issue, hope that it can help!

acrollet’s picture

I have applied this patch to the 2.x branch and created a dev release - please test and let me know if you find any problems.

acrollet’s picture

Status: Needs review » Fixed

Marking as fixed.

Status: Fixed » Closed (fixed)

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