While evaluating this module I noticed that on .info file, 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...

CommentFileSizeAuthor
#3 image-with-title.png14.24 KBdani3lr0se
#2 capitalize-module-name-2981268-2.patch293 byteslegaudinier

Comments

legaudinier created an issue. See original summary.

legaudinier’s picture

legaudinier commented
Assigned: legaudinier » Unassigned
Status: Active » Needs review
StatusFileSize
new capitalize-module-name-2981268-2.patch293 bytes

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

dani3lr0se’s picture

dani3lr0se commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new image-with-title.png14.24 KB

Patch applies clean with simplytest.me. Module title is now capitalized. See attached screenshot. Thanks for the patch @legaudinier. :)

pratik.mehta19’s picture

pratik.mehta19 commented
Version: 8.x-1.x-dev » 8.x-1.2
Status: Reviewed & tested by the community » Fixed
volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented

Thankyou for all the work you do!!
I am encouraging team mates to help out on Drupal.org, could you go back and give Lindsay Gaudinier (legaudinier) a credit for her patcg? She took the time to test and make sure everything was correct.
The box next to her name just gets ticked and she gets a credit! I see it as a way of incentive to help get involved it the community! It takes a village.

Cheers!

AmyJune - volkswagenchick

pratik.mehta19’s picture

pratik.mehta19 commented

Status: Fixed » Closed (fixed)

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