This would add a configuration link to the modules admin page.

CommentFileSizeAuthor
#2 2797935.patch396 bytesNancyDru
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

NancyDru created an issue. See original summary.

NancyDru’s picture

NancyDru
she/her/hers
Primary language English
Location Boston
CreditAttribution: NancyDru commented
FileSize
2797935.patch396 bytes

Here you go.

Status: Needs review » Needs work

The last submitted patch, 2: 2797935.patch, failed testing.

NancyDru’s picture

NancyDru
she/her/hers
Primary language English
Location Boston
CreditAttribution: NancyDru commented

I have no idea what a "CI error" is, but this patch does not affect executable code.

NancyDru’s picture

NancyDru
she/her/hers
Primary language English
Location Boston
CreditAttribution: NancyDru commented
Status: Needs work » Needs review
liquidcms’s picture

liquidcms CreditAttribution: liquidcms commented
Status: Needs review » Fixed

technically there isn't a config page for this module; the admin role config setting is added by core; not this module.

but nothing wrong with a little redundancy.. this has been committed to dev (the patch actually fails stating wrong format; but easy enough as its just the 1 line).

liquidcms’s picture

liquidcms CreditAttribution: liquidcms commented
Status: Fixed » Closed (fixed)