Skip to content

Conversation

@VitaliyBoyko
Copy link
Contributor

Original PR: #13318

Description

Fix typo in database column comment

@VitaliyBoyko VitaliyBoyko added backport partners-contribution Pull Request is created by Magento Partner Partner: Atwix Pull Request is created by partner Atwix labels May 17, 2018
@DanielRuf DanielRuf self-requested a review May 17, 2018 21:01
@VladimirZaets
Copy link
Contributor

Hi, @VitaliyBoyko, Your fix introduce changes in the DB schema. Due to Magento backward-compatible guide we can't make changes to DB schema in 2.1 Magento version.

@VladimirZaets VladimirZaets self-assigned this Jun 1, 2018
@DanielRuf
Copy link
Contributor

@vladimir-k also not with modifyColumn?

I also see no mention of schema name changes (or what is the last parameter?).
https://devdocs.magento.com/guides/v2.1/contributor-guide/backward-compatible-development/#db-schema

@VladimirZaets
Copy link
Contributor

@DanielRuf yes, you are right, the last parameter it's only comment. Our tests are failed on this fix, so I closed this PR, but in the current case I agree, we can merge it.

@magento-engcom-team magento-engcom-team merged commit bf18e7a into magento:2.1-develop Jun 7, 2018
@magento-engcom-team
Copy link
Contributor

Hi @VitaliyBoyko. Thank you for your contribution.
Changes from your Pull Request will be available with the upcoming 2.1.15 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Cleanup Component: Catalog Partner: Atwix Pull Request is created by partner Atwix partners-contribution Pull Request is created by Magento Partner Progress: accept Release Line: 2.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants