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 without_patch_block_cache_metadata.png17.8 KBkrina.addweb
#3 with_patch_block_cache_metadata.png17.82 KBkrina.addweb
#2 block_cache_metadata-info-3057121-2.patch320 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
block_cache_metadata-info-3057121-2.patch320 bytes

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

krina.addweb’s picture

krina.addweb CreditAttribution: krina.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Reviewed & tested by the community
FileSize
with_patch_block_cache_metadata.png17.82 KB
without_patch_block_cache_metadata.png17.8 KB

@alonaoneill, Thanks for the patch. Your patch is working fine as I tested the mentioned change using simplytest.me over Extend page. PFA

  • Insasse committed 1eeb83c on 8.x-1.x authored by alonaoneill 
    Issue #3057121 by alonaoneill, krina.addweb: Capitalize the name of the...
DiDebru’s picture

DiDebru
Primary language German
CreditAttribution: DiDebru at UEBERBIT GmbH commented
Status: Reviewed & tested by the community » Fixed

Looks good thanks for the patch committed.

Status: Fixed » Closed (fixed)

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