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
#4 subscription_entity-info-3079902-4.patch1.04 KBalonaoneill
#2 subscription_entity-info-3079902-2.patch1.04 KBalonaoneill

Comments

alonaoneill created an issue. See original summary.

alonaoneill’s picture

alonaoneill commented
Assigned: alonaoneill » Unassigned
Status: Active » Needs review
StatusFileSize
new subscription_entity-info-3079902-2.patch1.04 KB
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs review » Needs work

The Admin Toolbar module is not a Drupal core module.

alonaoneill’s picture

alonaoneill commented
StatusFileSize
new subscription_entity-info-3079902-4.patch1.04 KB

Yeah. You're right. Fixed it!
Thanks!

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs work » Reviewed & tested by the community

I would say it's good to go, now.