Skip to content

Backport of MAGETWO-59256 for 2.1: Custom composer modules break Component Manager #6718 #9692

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jun 27, 2017

Conversation

JTimNolan
Copy link

@JTimNolan JTimNolan commented May 18, 2017

Description

Backport of issue MAGETWO-59256 for 2.1
Cherry picked commits from develop, rebased, and squashed. Then removed some code from farther up the branch to make compatible with 2.1
First time making a PR for this so if I need to make changes/fix let me know.

Commits from the develop branch:

Fixed Issues (if relevant)

  1. Custom composer modules break Component Manager #6718: Custom composer modules break Component Manager

Manual testing scenarios

  1. Ensure Magento Marketplace keys have been added to system manager
  2. Require a module with composer (I used semaio/magento2-configimportexport)
  3. Go to /setup/index.php#/component-grid
  4. AJAX request to /setup/index.php/componentGrid/components no longer has an error

Bohdan Korablov and others added 3 commits May 18, 2017 11:50
magento#6718

(cherry picked from commit fc99447)

MAGETWO-59256: [GitHub] Custom composer modules break Component Manager magento#6718

(cherry picked from commit a9697c6)

MAGETWO-59256: [GitHub] Custom composer modules break Component Manager magento#6718

(cherry picked from commit 7352b7c)

MAGETWO-59256: [GitHub] Custom composer modules break Component Manager magento#6718

(cherry picked from commit e871276)

MAGETWO-59256: [GitHub] Custom composer modules break Component Manager magento#6718

(cherry picked from commit 01f1f42)

MAGETWO-59256: [GitHub] Custom composer modules break Component Manager magento#6718

(cherry picked from commit e072d65)
@magento-cicd2
Copy link
Contributor

magento-cicd2 commented May 18, 2017

CLA assistant check
All committers have signed the CLA.

@JTimNolan
Copy link
Author

@shiftedreality Could you take a look at this? In reference to #6718 which you closed.

@shiftedreality
Copy link
Member

@JTimNolan thanks for pointing this.
Assigned to Community Engineering member for PR proessing

@JTimNolan
Copy link
Author

@shiftedreality @maghamed Do we have an ETA for this?

@ishakhsuvarov ishakhsuvarov self-assigned this Jun 23, 2017
@ishakhsuvarov ishakhsuvarov added this to the June 2017 milestone Jun 23, 2017
@magento-team magento-team merged commit 9a65346 into magento:2.1-develop Jun 27, 2017
magento-team pushed a commit that referenced this pull request Jun 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants