Apply new {project}:{module} format for dependencies in info.yml https://www.drupal.org/docs/8/creating-custom-modules/let-drupal-8-know-about-your-module-with-an-infoyml-file
My 2 cents.

CommentFileSizeAuthor
#6 interdiff-2998835-2-6.txt184 bytesmrinalini9
#6 Update_dependencies_format_in_info.yml_2998835-6.patch384 bytesmrinalini9
#3 Menu Admin per Menu_Dependencies in .info.png19.1 KBakash.addweb
#2 Update_dependencies_format_in_info.yml_2998835-2.patch354 byteschipway

Comments

chipway created an issue. See original summary.

chipway’s picture

chipway commented
Assigned: chipway » Unassigned
Status: Needs work » Needs review
StatusFileSize
new Update_dependencies_format_in_info.yml_2998835-2.patch354 bytes

Proposed patch.

akash.addweb’s picture

akash.addweb commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Menu Admin per Menu_Dependencies in .info.png19.1 KB

@chipway, Thanks for providing the patch, Your patch clears all the dependency errors that shown in Preview report. I follow the report that I attached below.

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Status: Reviewed & tested by the community » Needs work

This module is also missing a dependency on menu_link_content. We can probably add it here since we're updating the .info.yml file anyway.

mrinalini9’s picture

mrinalini9 commented
Assigned: Unassigned » mrinalini9
mrinalini9’s picture

mrinalini9 commented
Assigned: mrinalini9 » Unassigned
Status: Needs work » Needs review
StatusFileSize
new Update_dependencies_format_in_info.yml_2998835-6.patch384 bytes
new interdiff-2998835-2-6.txt184 bytes

Updated patch and added dependency on menu_link_content as mentioned in #4, please review.

jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Status: Needs review » Reviewed & tested by the community
jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented

  • JeroenT committed ece5a8f on 8.x-1.x authored by mrinalini9 
    Issue #2998835 by mrinalini9, chipway, Ronak.addweb, JeroenT: Update...
jeroent’s picture

jeroent
Primary language Dutch
Location 🇧🇪
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x-1.x. Thanks!

Status: Fixed » Closed (fixed)

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