Problem/Motivation

index_now_commerce was removed from this repository because I couldn't find a way to install the commerce module dependency from the submodule and tests were failing because of it.
However, having two separate modules makes maintenance more complicated.

Steps to reproduce

Proposed resolution

Try one or both of the following:

  • composer require drupal/commerce --dev from the main module
  • Install it within the CI
    composer:
  before_script:
    - composer require drupal/commerce --no-update

Remaining tasks

  • Move index_now_commerce code in the modules directory
  • Ensure that the tests pass

User interface changes

API changes

Data model changes

Issue fork index_now-3577759

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

macsim created an issue. See original summary.

macsim opened merge request !39

  • macsim committed af606166 on 3.1.x 
    feat: #3577759 Include index_now_commerce as a submodule
macsim’s picture

macsim commented
Status: Active » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

macsim’s picture

macsim commented
Status: Fixed » Closed (fixed)