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

Fix issue #10565 #10575

Merged
merged 4 commits into from
Aug 19, 2017
Merged

Fix issue #10565 #10575

merged 4 commits into from
Aug 19, 2017

Conversation

thlassche
Copy link
Contributor

@thlassche thlassche commented Aug 17, 2017

Fix: Creation of a new product with custom attribute set now works again

Description

85db4c8#diff-620105b6e833499c14f4eb74ce8a4532 introduced some changes wich broke the add new product screen when choosing a custom attribute set with a multiselect attribute in it. This pull request fixes that

Fixed Issues

  1. Magento ver. 2.1.8 New Product with Custom attribute set not working #10565: Magento ver. 2.1.8 New Product with Custom attribute set not workink

Manual testing scenarios

  1. Create a new product from admin, select a non-default attribute set
  2. Check that the product form loads without JS errors

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

@thlassche
Copy link
Contributor Author

Please also backport this in the 2.1.x branch. This is also broken on the production release 2.1.8

@ishakhsuvarov
Copy link
Contributor

ishakhsuvarov commented Aug 17, 2017

@thlassche It would be nice to cover this case with a JS test. You may just update the existing one.

@thlassche
Copy link
Contributor Author

@ishakhsuvarov Ok, will try to find some time this evening

@ishakhsuvarov
Copy link
Contributor

@thlassche Thanks a lot. Please let us know if you need any help with it.

@ishakhsuvarov ishakhsuvarov self-assigned this Aug 17, 2017
@ishakhsuvarov ishakhsuvarov added this to the August 2017 milestone Aug 17, 2017
@thlassche
Copy link
Contributor Author

@ishakhsuvarov Just added the tests

@ishakhsuvarov
Copy link
Contributor

@thlassche Thank you

@magento-team magento-team merged commit 02c018d into magento:develop Aug 19, 2017
magento-team pushed a commit that referenced this pull request Aug 19, 2017
magento-team pushed a commit that referenced this pull request Aug 19, 2017
[EngCom] Public Pull Requests
 - MAGETWO-71743: Removed instantiating new driverpool class when it's already available. #10590
 - MAGETWO-71742: Changed typos in class comment #10589
 - MAGETWO-71705: fixed notice when remove column from grid #10579
 - MAGETWO-71670: Fix issue #10565 #10575
magento-team pushed a commit that referenced this pull request Oct 10, 2017
magento-team pushed a commit that referenced this pull request Oct 10, 2017
Fixed issues:
- MAGETWO-71552: Attribute values on store view level not searchable - for 2.2
- MAGETWO-72866: Redundant indexers invalidation - RIATCS-340
- MAGETWO-75458: [Backport] - Fix overwrite default value image/file with NULL #10253 - for 2.2
- MAGETWO-75460: [Backport] - LowestPriceOptionsProvider returns products without attributes which are used for price calculation (e.g. tax adjustment)
- MAGETWO-80193: [2.2.x] - Add cast to string for CUST_GROUP_ALL #10475
- MAGETWO-80204: [2.2.x] - Grammar fix for #9533 #10627
- MAGETWO-71549: Impossible to export Advanced Prices on a medium profile
- MAGETWO-80198: [2.2.x] - Fix issue #10565 #10575
- MAGETWO-80197: [2.2.x] - Fix JS translation search #10445
- MAGETWO-80195: [2.2.x] - Send different base currency in Google analytics #10508
hostep pushed a commit to hostep/magento2 that referenced this pull request Feb 7, 2018
magento-engcom-team pushed a commit that referenced this pull request Feb 21, 2018
…10565 #10575 #13549

 - Merge Pull Request #13549 from hostep/magento2:backport-magetwo-80198
 - Merged commits:
   1. d95e15c
magento-engcom-team added a commit that referenced this pull request Feb 21, 2018
Accepted Public Pull Requests:
 - #13549: Backport of MAGETWO-80198 for Magento 2.1: Fix issue #10565 #10575 (by @hostep)
 - #13543: Backport of MAGETWO-69379 for Magento 2.1: use payment method name to� (by @hostep)


Fixed GitHub Issues:
 - #10565: Magento ver. 2.1.8 New Product with Custom attribute set not working (reported by @gilbertsohal) has been fixed in #13549 by @hostep in 2.1-develop branch
   Related commits:
     1. d95e15c

 - #6207: Checkbox IDs for Terms and Conditions should be unique in Checkout (reported by @bka) has been fixed in #13543 by @hostep in 2.1-develop branch
   Related commits:
     1. 6ec9e22
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.

3 participants