"Capitalize the name of the module, because module names are proper nouns."

https://www.drupal.org/docs/develop/documenting-your-project/help-text-s...

I also found some grammar issues in the description on the module in this same file.

Thanks, patch to follow!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

Assigned: volkswagenchick » Unassigned
Status: Needs work » Needs review
FileSize
537 bytes
9.18 KB
10.89 KB

I uploaded a patch that capitalizes the name of the module and fixes some grammar issues in the .info file.

Thanks so much for this module!!

dani3lr0se’s picture

Status: Needs review » Reviewed & tested by the community

Patch applies cleanly with simplytest.me. Grammar looks good and makes sense. Thanks for the patch.

StijnStroobants’s picture

Thanks for the patch! Committed!

StijnStroobants’s picture

Status: Reviewed & tested by the community » Fixed
volkswagenchick’s picture

Thankyou for all the work you do!!
I am encouraging team mates to help out on Drupal.org, could you go back and give daniel_rose a credit for his RTBC?? The box next to his name just gets ticked and he 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

Status: Fixed » Closed (fixed)

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