Capitalize the name of the module, because module names are proper nouns.
Also according to Drupal standards, modules should include dependencies in the .info.yml file.
Dependencies should be namespaced in the format {project}:{module}, where {project} is the project name as it appears in the Drupal.org URL (e.g. drupal.org/project/views) and {module} is the module's machine name.
https://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-...
Patch to follow, thanks!

CommentFileSizeAuthor
#3 without_patch_simple_global_filter.png24.82 KBAanal.addweb
#3 with_patch_simple_global_filter.png25 KBAanal.addweb
#3 dependency_simple_global_filter.png31.52 KBAanal.addweb
#2 simple_global_filter-info-3064677-2.patch412 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
simple_global_filter-info-3064677-2.patch412 bytes
Aanal.addweb’s picture

Aanal.addweb CreditAttribution: Aanal.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Reviewed & tested by the community
FileSize
dependency_simple_global_filter.png31.52 KB
with_patch_simple_global_filter.png25 KB
without_patch_simple_global_filter.png24.82 KB

@alonaoneill, Thanks for the patch. Your patch is working fine. I tested it using simplytest.me over Extend page and pareview.sh. PFA

  • eldrupalista committed 74056df on 8.x-1.x 
    Issue #3064677 by alonaoneill, Aanal.addweb: Adjusted info.yml to Drupal...
eldrupalista’s picture

eldrupalista CreditAttribution: eldrupalista at Exove commented
Status: Reviewed & tested by the community » Fixed

Done.
Thank you Alona & Aanal for the help.

Status: Fixed » Closed (fixed)

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