Skip to content

Conversation

tzyganu
Copy link
Contributor

@tzyganu tzyganu commented Mar 3, 2016

No description provided.

@mazhalai
Copy link
Contributor

mazhalai commented Mar 8, 2016

@tzyganu Please sync with develop. Builds have been fixed.

@hshar7
Copy link
Contributor

hshar7 commented Mar 8, 2016

Internal ticket: MAGETWO-50238

@hshar7 hshar7 added the Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development label Mar 8, 2016
*/
public function upgrade(SchemaSetupInterface $setup, ModuleContextInterface $context)
{
$setup->startSetup();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

start/end-Setup might not be necessary here. Better to not use it, if it's not needed. It disables FK validation.

@strajeru
Copy link
Contributor

@mazhalai can you please restart the tests. some are failing with The command "sudo apt-get update -qq" failed and exited with 100 during ..

@mazhalai
Copy link
Contributor

@strajeru Builds restarted.

@magento-team magento-team merged commit fc7c06d into magento:develop May 3, 2016
@okorshenko
Copy link
Contributor

@strajeru Thank you for contribution. Your Pull Request merged to develop branch.

magento-engcom-team pushed a commit that referenced this pull request Jan 26, 2019
[EngCom] Public Pull Requests - GraphQL
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants