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!

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

Status: Active » Needs review
StatusFileSize
new391 bytes

I uploaded a patch that capitalizes the name of the module in the .info file to align with Drupal standards, thanks.

volkswagenchick’s picture

StatusFileSize
new1.58 KB

Upon further investigation, we found other instances where the name should be in title case. New patch uploaded.

Thanks!

daggerhart’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new6.77 KB
new31.22 KB
new14.55 KB

This 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_token and admin/config/content/access_unpublished

chr.fritsch’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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