While evaluating this module I noticed that on the Extend page, the project name is in lower case and should be in title case to be consistent with the rest of the project names on the page.
"Capitalize the name of the module, because module names are proper nouns."
https://www.drupal.org/docs/develop/documenting-your-project/help-text-s...
Patch to follow, thanks!
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | access_unpublished-access-tokens.png | 14.55 KB | daggerhart |
| #4 | access_unpublished-configuration.png | 31.22 KB | daggerhart |
| #4 | access_unpublished-module-title.png | 6.77 KB | daggerhart |
| #3 | access_unpublished-capitalize-name-2985352-3.patch | 1.58 KB | volkswagenchick |
Comments
Comment #2
volkswagenchickI uploaded a patch that capitalizes the name of the module in the .info file to align with Drupal standards, thanks.
Comment #3
volkswagenchickUpon further investigation, we found other instances where the name should be in title case. New patch uploaded.
Thanks!
Comment #4
daggerhart commentedThis looks good. Patch applies cleanly. I spun the patch up on simplytest and UI is updated on the modules list page, and route titles are updated on
admin/content/access_tokenandadmin/config/content/access_unpublishedComment #6
chr.fritsch