Skip to content

Conversation

@NadiyaS
Copy link
Contributor

@NadiyaS NadiyaS commented Dec 12, 2019

Description

Existed patch for 2.3.3 - MC-21696, did not solve whole problem with ES and it causes another bug.
Instead of this patch was created new one based on 2 fixes from 2.3.4-develop branch:

Manual testing scenarios

  1. test for https://jira.corp.magento.com/browse/MC-22852
  2. test for https://jira.corp.magento.com/browse/MC-19527

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)

@NadiyaS NadiyaS added the onhold label Dec 12, 2019
patches.json Outdated
},
"Catalog Product pagination does not work on Elasticsearch 6.x": {
"2.3.3": "MC-21696__fix_broken_pagination_on_ES6__2.3.3.patch"
"2.3.3": "MAGECLOUD-4847__Elasticsearch_and_products_display_fix__2.3.3.patch"
Copy link
Member

Choose a reason for hiding this comment

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

We mostly use lowercase for patches, let's keep the same naming convention here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

changed

@arhiopterecs arhiopterecs added qa and removed Progress: review PR/issue status labels Dec 18, 2019
@andriyShevtsov andriyShevtsov self-assigned this Dec 18, 2019
@andriyShevtsov
Copy link
Collaborator

QA approved

@mveeramneni mveeramneni added Progress: approved PR/issue status and removed qa labels Jan 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Progress: approved PR/issue status

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants