Hi,

Would it be possible to make permissions more granular so that site editors can have access to admin/commerce/config/email to update content of the emails without going as far as setting "Configure store"?

If i get time I'll try and supply a patch for this myself.

Thx

CommentFileSizeAuthor
#3 commerce_email-Permission-level-granularity-1559264-3.patch1.26 KBszeidler
#1 0001-fixed-1559264-Permission-level-granularity.patch1.24 KBgeek-merlin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

geek-merlin’s picture

geek-merlin
they/omg
Primary language German
Location Freiburg, Germany
CreditAttribution: geek-merlin commented
Status: Active » Needs review
FileSize
0001-fixed-1559264-Permission-level-granularity.patch1.24 KB

trivial patch flying in.

you may need a current variable.module for the permission inheritance to work, and remember to clear cache.

absoludo’s picture

absoludo CreditAttribution: absoludo commented

Simple but efficient patch.
We really needed the option to allow access per role.

Please implement this feature on the next release.

szeidler’s picture

szeidler CreditAttribution: szeidler commented
Issue summary: View changes
FileSize
commerce_email-Permission-level-granularity-1559264-3.patch1.26 KB

Thanks Axel. We also need it in commerce_email_menu to make the page visible :) Attached patch.

rszrama’s picture

rszrama CreditAttribution: rszrama at Centarro commented

Reviewing, seems simple enough.

rszrama’s picture

rszrama CreditAttribution: rszrama at Centarro commented
Status: Needs review » Fixed

Committed with whitespace fixes.

  • rszrama committed 2207507 on 7.x-2.x authored by geek-merlin 
    Issue #1559264 by szeidler, geek-merlin, rszrama: Permission level...

Status: Fixed » Closed (fixed)

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