When installing the module using composer: composer require drupal/seo_checklist, only seo_checklist is downloaded. The composer.json file needs to be updated to also require drupal/checklistapi

Patch to follow.

CommentFileSizeAuthor
#2 seo_checklist-composer-require-checklistapi-3174100-2.patch435 bytesthejimbirch
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

thejimbirch created an issue. See original summary.

thejimbirch’s picture

thejimbirch CreditAttribution: thejimbirch at Kanopi Studios commented
Assigned: thejimbirch » Unassigned
Status: Active » Needs review
FileSize
seo_checklist-composer-require-checklistapi-3174100-2.patch435 bytes

Patch attaches that adds the require section to the composer.json file.

TravisCarden’s picture

TravisCarden CreditAttribution: TravisCarden as a volunteer commented
Status: Needs review » Postponed (maintainer needs more info)

Thanks for the patch, @thejimbirch! It was definitely correct and needed. However, I'm not confident it will fix your problem because I was unable to reproduce it in the first place.

$ composer require drupal/seo_checklist
Using version ^4.1 for drupal/seo_checklist
./composer.json has been updated
Gathering patches for root package.
No patches supplied.
Loading composer repositories with package information
Updating dependencies (including require-dev)
Package operations: 2 installs, 0 updates, 0 removals
Gathering patches for root package.
No patches supplied.
Gathering patches for dependencies. This might take a minute.
  - Installing drupal/checklistapi (2.0.0): Loading from cache
  - Installing drupal/seo_checklist (4.1.0): Loading from cache
Writing lock file
Generating autoload files

I would hate to make a release that didn't even work for you. Can you give me steps to reproduce?

thejimbirch’s picture

thejimbirch CreditAttribution: thejimbirch at Kanopi Studios commented

hmm, I was helping someone in the Drupal Slack: https://drupal.slack.com/archives/C54EZA71T/p1601472184007300

Let me try to fire up a clean install to see.

thejimbirch’s picture

thejimbirch CreditAttribution: thejimbirch at Kanopi Studios commented

Yeah, it installed both for me on a clean install. I asked the person in slack to try again and follow up here.

thejimbirch’s picture

thejimbirch CreditAttribution: thejimbirch at Kanopi Studios commented
Status: Postponed (maintainer needs more info) » Needs review

They posted back in slack that they were using Composer 2. They did it again with the dev branch and it did update.

https://drupal.slack.com/archives/C54EZA71T/p1601565144016300?thread_ts=...

TravisCarden’s picture

TravisCarden CreditAttribution: TravisCarden as a volunteer commented
Status: Needs review » Fixed

Great. It sounds like the issue is solved then, right? 👍

Status: Fixed » Closed (fixed)

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