It would be preferable for some sites if this had its own permission.

CommentFileSizeAuthor
#2 site_verify-add_permission-1898908-2.patch2.37 KBjdillick

Comments

jdillick’s picture

jdillick commented
Title: It's own permission » Its own permission
jdillick’s picture

jdillick commented
Status: Needs work » Needs review
StatusFileSize
new site_verify-add_permission-1898908-2.patch2.37 KB

Here is a proposed patch.

jdillick’s picture

jdillick commented
Category: bug » feature
jibize’s picture

jibize commented

Yes plus one for this feature.

Patch works great for me, thanks!

quantos’s picture

quantos commented

hi guys. I'm looking to add permissions to a new SEO role I've created on various sites. Can you confirm that's what this patch/issue is (please)? Will this also be available in future releases? Many thanks, in advance.

Q.

jdillick’s picture

jdillick commented

hi guys. I'm looking to add permissions to a new SEO role I've created on various sites. Can you confirm that's what this patch/issue is (please)? Will this also be available in future releases? Many thanks, in advance.

The current version will work if you give your SEO role "administer site configuration". That permission is overly broad for some organizations, who would like to give access to site verify without giving access to everything else that "administer site configuration" gives. This patch will enable you to give *only* access to site verify to your SEO role, yes.

John

quantos’s picture

quantos commented

#6 many thanks John. Much appreciated.

Q.

stewart.adam’s picture

stewart.adam commented
Status: Needs review » Reviewed & tested by the community

#2 works for me! Thanks.

roam2345’s picture

roam2345 commented

works here as well +1

vladsavitsky’s picture

vladsavitsky commented

Patch works fine. Code is pretty simple and clear. Why should we wait? Please commit this patch because it takes a lot extra time to manage patches. Thanks.

joachim’s picture

joachim commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Thanks for the patch! Committed and pushed; will be in new release soon.

git commit -m "Issue #1898908 by jdillick: Added user permission for verifications admin." --author="jdillick "

  • Commit e7b73fb on 7.x-1.x authored by jdillick, committed by joachim: 
    Issue #1898908 by jdillick: Added user permission for verifications...

Status: Fixed » Closed (fixed)

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