Skip to content
This repository has been archived by the owner on Apr 29, 2019. It is now read-only.

Replace ZF1 Zend_Feed with ZF2/3 Zend\Feed #89

Merged
merged 1 commit into from
Mar 15, 2018

Conversation

bbatsche
Copy link
Contributor

@bbatsche bbatsche commented Mar 12, 2018

Description

Replacing ZF1 Zend_Feed with Zend\Feed allows us to eliminate it as a module from Zend framework, and opens up many other additional modules that were otherwise dependencies on it.

Fixed Issues (if relevant)

  1. Eliminate usage of Zend_Feed from Magento 2 Open Source #70: Eliminate usage of Zend_Feed from Magento 2 Open Source

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)

@bbatsche bbatsche requested a review from buskamuza March 12, 2018 16:20
@@ -4,7 +4,7 @@
"Read more about it at https://getcomposer.org/doc/01-basic-usage.md#composer-lock-the-lock-file",
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use composer update <package>=<version> instead of bare composer update. At least one issue I already observed is new version of zend-code behaves differently. We might do full upgrade as a separate task.

@buskamuza
Copy link
Contributor

buskamuza commented Mar 13, 2018

  • Please update EE composer.json as well
  • Please check composer install on PHP 7.0. Right now Travis build fails on PHP 7.0

@magento-engcom-team
Copy link
Collaborator

@bbatsche thank you for contributing. Please accept Community Contributors team invitation here to gain extended permissions for this repository.

@magento-engcom-team magento-engcom-team merged commit b73d1c8 into 2.3-develop Mar 15, 2018
magento-engcom-team pushed a commit that referenced this pull request Jun 21, 2018
…-Category

[architects] MAGETWO-92773: [GraphQL] Products cannot be fetched in parent/anchor category #89
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants