Skip to content
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

Country of Manufacture drop down is limited:issue:4579 #4599

Closed
wants to merge 1 commit into from
Closed

Country of Manufacture drop down is limited:issue:4579 #4599

wants to merge 1 commit into from

Conversation

sivajik34
Copy link
Contributor

Steps to reproduce

Configure M2 config to only sell to 1 (or 2 or 3 etc) countries.
Click on "country of manufacture" drop-down in product profile.

Expected result

A list of all countries (regardless of which countries we want to sell to) should be displayed

Actual result

Country list is restricted to the "Allowed Countries" list, as per the site configuration.

This above problem does not manifest in Magento 1 - it works as expected (i.e.: even if we have configured the site only to sell to "France", we can configure a product to be manufactured by any country).

@sivajik34
Copy link
Contributor Author

I'm not able to signup Contributor License Agreement.

@sivajik34
Copy link
Contributor Author

phpunit $TEST_FILTER

PHPUnit 4.1.0 by Sebastian Bergmann.

Configuration read from /home/travis/build/magento/magento2/dev/tests/integration/phpunit.xml

PHP Parse error: syntax error, unexpected '.', expecting ',' or ';' in /home/travis/build/magento/magento2/lib/internal/Magento/Framework/Composer/Test/Unit/ComposerFactoryTest.php on line 17

Parse error: syntax error, unexpected '.', expecting ',' or ';' in /home/travis/build/magento/magento2/lib/internal/Magento/Framework/Composer/Test/Unit/ComposerFactoryTest.php on line 17

@paliarush
Copy link
Contributor

Fixed in scope of ccf566e

@paliarush paliarush closed this Aug 3, 2016
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.

None yet

4 participants