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

Backport of PR-8246 for Magento 2.1: Fixes #7723 - saving multi selec… #13546

Merged
merged 1 commit into from
Mar 5, 2018

Conversation

hostep
Copy link
Contributor

@hostep hostep commented Feb 7, 2018

…t field in UI component form

(cherry picked from commit ca9e33f)

Use underscore to fetch selected values as array

(cherry picked from commit 16170ce)

Description

This is a backport of #8246 for Magento 2.1

Fixed Issues (if relevant)

  1. Catalog rule contains-condition not saving multiple selection in 2.1.2 #7723: Catalog rule contains-condition not saving multiple selection in 2.1.2

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)

…i select field in UI component form

(cherry picked from commit ca9e33f)

Use underscore to fetch selected values as array

(cherry picked from commit 16170ce)
@osrecio osrecio self-assigned this Feb 7, 2018
@osrecio osrecio added this to the February 2018 milestone Feb 7, 2018
@osrecio
Copy link
Member

osrecio commented Feb 7, 2018

Thanks @hostep I'm working to process this PR. Thanks for the Backport!

@okorshenko okorshenko modified the milestones: February 2018, March 2018 Mar 1, 2018
@osrecio osrecio removed their assignment Mar 5, 2018
@magento-engcom-team magento-engcom-team merged commit a114725 into magento:2.1-develop Mar 5, 2018
magento-engcom-team pushed a commit that referenced this pull request Mar 5, 2018
magento-engcom-team pushed a commit that referenced this pull request Mar 5, 2018
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.

6 participants