Problem/Motivation

I want to use Acquia Drupal Starterkit Document module for remote Document media but it comes with Acquia Drupal Starterkit Common module with dozens of other unnecessary modules, which enabled dozens of other unnecessary modules which not required for my application.

Steps to reproduce

  • Require Acquia Drupal Starterkit Document module
  • Install Acquia Drupal Starterkit Document module
  • Verify dozens of other unnecessary modules installed

Proposed resolution

To be able to use the Acquia Drupal Starterkit Document module without having to enable the unnecessary modules that come with the Acquia Drupal Starterkit Common module, and to have a more streamlined and efficient application.

Remaining tasks

  • Review and limit module dependencies so that only modules essential to Acquia CMS Image remain enabled.
  • Update the GitLab CI template and use CI template provided by Drupal Core, replacing the current ORCA setup, and remove any files specific to ORCA.
  • Add / Update code in Acquia CMS Image module for module to function as expected.
  • Refactor and update PHPUnit tests to ensure they are passing.
  • Ensure all CI pipelines complete successfully, including those in the "allowed failures" category.

User interface changes

No

API changes

No

Data model changes

No

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:

Comments

rajeshreeputra created an issue. See original summary.

rajeshreeputra’s picture

Status: Active » Fixed

completed as part of PR.

Status: Fixed » Closed (fixed)

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

vishalkhode’s picture

Status: Closed (fixed) » Needs work

Re-opening this back. As we'll complete the development and merge as part of this ticket.

vishalkhode’s picture

Issue summary: View changes
vishalkhode’s picture

Issue summary: View changes
vishalkhode’s picture

Status: Needs work » Needs review
ankitv18’s picture

Left a comment, please check.
Keeping this in review for other inputs

rajeshreeputra’s picture

Status: Needs review » Reviewed & tested by the community

Looks good

ankitv18’s picture

Ignore above feedback as Backward Compatibility is handled in the .module hooks

vishalkhode’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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