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 vault_payment_token install script type where column defaults were not set #12965

Merged
merged 1 commit into from
Jan 10, 2018
Merged

Fix vault_payment_token install script type where column defaults were not set #12965

merged 1 commit into from
Jan 10, 2018

Conversation

lukeconvertdigital
Copy link

Fixed typo in the vault_payment_token install script.

Description

The is_active and is_visible columns were defaulting to false, however should default to true.

@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Jan 3, 2018

CLA assistant check
All committers have signed the CLA.

@joni-jones joni-jones self-assigned this Jan 3, 2018
@joni-jones joni-jones self-requested a review January 3, 2018 09:15
@joni-jones joni-jones removed their assignment Jan 3, 2018
@ishakhsuvarov ishakhsuvarov self-assigned this Jan 3, 2018
@ishakhsuvarov ishakhsuvarov added this to the January 2018 milestone Jan 3, 2018
magento-team pushed a commit that referenced this pull request Jan 8, 2018
magento-team pushed a commit that referenced this pull request Jan 8, 2018
… script type where column defaults were not set (PR #12965 backport 2.1) #1149

 - Merge Pull Request magento-engcom/magento2ce#1149 from p-bystritsky/magento2:PULL-12965-BACKPORT
 - Merged commits:
   1. 3230447
   2. 634afa8
magento-team pushed a commit that referenced this pull request Jan 8, 2018
… script type where column defaults were not set (PR #12965 backport 2.1) #1149
magento-team pushed a commit that referenced this pull request Jan 8, 2018
Accepted Public Pull Requests:
 - magento-engcom/magento2ce#1149: magento/magento2#NONE: Fix vault_payment_token install script type where column defaults were not set (PR #12965 backport 2.1) (by @p-bystritsky)
 - #12958: [Backport to 2.1-develop] Fix #9243 - Upgrade ZF components. Zend_Service (by @dverkade)
 - #12959: [Backport to 2.1-develop] Fix #10812: htaccess Options override (by @dverkade)
 - #12930: Fix wishlist item getBuyRequest with no options (by @jameshalsall)


Fixed GitHub Issues:
 - #9243: Upgrade ZF components. Zend_Service (reported by @okorshenko) has been fixed in #12958 by @dverkade in 2.1-develop branch
   Related commits:
     1. abc7bc6

 - #10812: htaccess Options override (reported by @s3rj1k) has been fixed in #12959 by @dverkade in 2.1-develop branch
   Related commits:
     1. 704960a
@magento-team magento-team merged commit fb9d8d6 into magento:2.2-develop Jan 10, 2018
magento-team pushed a commit that referenced this pull request Jan 10, 2018
magento-team pushed a commit that referenced this pull request Jan 10, 2018
…ere column defaults were not set (PR #12965 forwardport)
magento-team pushed a commit that referenced this pull request Jan 10, 2018
… script type where column defaults were not set (PR #12965 forwardport 2.3) #1150

 - Merge Pull Request magento-engcom/magento2ce#1150 from p-bystritsky/magento2:PULL-12965-FORWARDPORT
 - Merged commits:
   1. 9e1610e
   2. 8e3bbd0
   3. 7da8a50
   4. c76f107
magento-team pushed a commit that referenced this pull request Jan 10, 2018
… script type where column defaults were not set (PR #12965 forwardport 2.3) #1150
magento-team added a commit that referenced this pull request Jan 10, 2018
Accepted Public Pull Requests:
 - #12937: [Backport 2.3-develop] Handle multiple errors in customer address validation when shown in adminhtml customer edit page (by @adrian-martinez-interactiv4)
 - magento-engcom/magento2ce#1150: magento/magento2#NONE: Fix vault_payment_token install script type where column defaults were not set (PR #12965 forwardport 2.3) (by @p-bystritsky)
 - magento-engcom/import-export-improvements#95: MAGETWO-49985: product custom options export in reverse order fixed (by @umarch-rltsquare)
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