While evaluating this module, I noticed some dependency namespacing errors in the .info 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.

Patch upload to follow.

CommentFileSizeAuthor
#2 photoshelter-dependency-fix-2990510-2.patch307 byteslegaudinier

Comments

legaudinier created an issue. See original summary.

legaudinier’s picture

legaudinier commented
Assigned: legaudinier » Unassigned
Status: Active » Needs review
StatusFileSize
new photoshelter-dependency-fix-2990510-2.patch307 bytes

I created and upload a patch that fixes the dependency errors in the info.yml file. Thanks!

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Status: Needs review » Reviewed & tested by the community

Code looks good, the dependency namespacing now aligns with drupal documentation standards.

Patch applied using simplytest.me. Marking RTBC

  • olivier.br committed 250f83b on 8.x-2.x 
    Issue #2990510 by legaudinier: Dependencies namespacing in .info file
olivier.br’s picture

olivier.br commented
Status: Reviewed & tested by the community » Fixed

Thanks for the patch !

Status: Fixed » Closed (fixed)

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