Problem/Motivation

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.

CommentFileSizeAuthor
#2 3330338-2.patch663 bytescharchil khandelwal

Issue fork store_locator_block-3330338

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Charchil Khandelwal created an issue. See original summary.

charchil khandelwal’s picture

charchil khandelwal commented
Assigned: charchil khandelwal » Unassigned
Status: Active » Needs review
StatusFileSize
new 3330338-2.patch663 bytes

Please review.

shubham rathore’s picture

shubham rathore commented

Hi patch#2 applies cleanly for me.
Thank You.

Dheeraj Jhamtani made their first commit to this issue’s fork.

Dheeraj Jhamtani opened merge request !4

shubham rathore’s picture

shubham rathore commented
Status: Needs review » Reviewed & tested by the community

Hi patch is applied properly and working as per requirement LGTM so moving to RTBC

heni_deepak’s picture

heni_deepak commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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