Capitalize the name of the module, because module names are proper nouns.
https://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-...
Patch to follow, thanks!

CommentFileSizeAuthor
#3 with_patch_Glossify.png49.53 KBAanal.addweb
#3 without_patch_Glossify.png46.41 KBAanal.addweb
#2 glossify-info-3008400-2.patch883 bytesalonaoneill
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alonaoneill created an issue. See original summary.

alonaoneill’s picture

alonaoneill CreditAttribution: alonaoneill at Hook 42 commented
Assigned: alonaoneill » Unassigned
Status: Active » Needs review
FileSize
glossify-info-3008400-2.patch883 bytes

I uploaded a patch that Capitalize the name of the submodules in the .info.yml file.
Thanks!

Aanal.addweb’s picture

Aanal.addweb CreditAttribution: Aanal.addweb at AddWeb Solution Pvt. Ltd. commented
FileSize
without_patch_Glossify.png46.41 KB
with_patch_Glossify.png49.53 KB

@alonaoneill, Thanks for providing the patch, Your patch is working fine for me as per the mentioned changes above. I checked both the changes with simplytest.me over extend page. PFA

govind.maloo’s picture

govind.maloo CreditAttribution: govind.maloo at Salsa Digital commented
Status: Needs review » Reviewed & tested by the community

  • keesje committed a601dc0 on 8.x-1.x authored by alonaoneill 
    Issue #3008400 by alonaoneill, Aanal.addweb: Capitalize the name of the...
keesje’s picture

keesje CreditAttribution: keesje at Media.Monks commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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