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.

Sign up for GitHub

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

Jump to bottom

Deleting disabled category forces category flat and catalog search re-indexed fully #32082

Merged
merged 12 commits into from
Mar 7, 2021
Merged

Deleting disabled category forces category flat and catalog search re-indexed fully #32082

merged 12 commits into from
Mar 7, 2021

Conversation

engcom-Charlie
Copy link
Contributor

@engcom-Charlie engcom-Charlie commented Feb 10, 2021 •
edited
Loading

Description (*)

Related Pull Requests

Fixed Issues (if relevant)

  1. Fixes Deleting disabled category forces category flat and catalog search re-indexed fully #23297

Manual testing scenarios (*)

  1. ...
  2. ...

Questions or comments

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 are green)

@engcom-Charlie
Copy link
Contributor Author

@magento run all tests

@magento magento deleted a comment from m2-assistant bot Feb 10, 2021
@engcom-Charlie
Copy link
Contributor Author

@magento run all tests

@danielrussob
Copy link

@magento give me 2.4-develop instance

@magento-deployment-service
Copy link

Hi @danielrussob. Thank you for your request. I'm working on Magento instance for you.

@danielrussob
Copy link

@magento give me test instance

@magento-deployment-service
Copy link

Hi @danielrussob. Thank you for your request. I'm working on Magento instance for you.

@magento-deployment-service
Copy link

Hi @danielrussob, here is your Magento Instance: https://65c3d1030c316eafdf0cd8cd74bf10e7.instances.magento-community.engineering
Admin access: https://65c3d1030c316eafdf0cd8cd74bf10e7.instances.magento-community.engineering/admin_bd1d
Login: 72522b5a
Password: 95bbf7fb195f

@magento-deployment-service
Copy link

Hi @danielrussob, here is your Magento Instance: https://65c3d1030c316eafdf0cd8cd74bf10e7-2-4-develop.instances.magento-community.engineering
Admin access: https://65c3d1030c316eafdf0cd8cd74bf10e7-2-4-develop.instances.magento-community.engineering/admin_e670
Login: 35d0ee90
Password: 09aeb4c5c997

@m2-community-project m2-community-project bot moved this from Pending Review to Review in Progress in High Priority Pull Requests Dashboard Feb 18, 2021
@engcom-Charlie
Copy link
Contributor Author

@magento run all tests

@m2-community-project m2-community-project bot moved this from Extended Testing (optional) to Ready for Testing in High Priority Pull Requests Dashboard Feb 23, 2021
@magento-engcom-team
Copy link
Contributor

Hi @aleron75, thank you for the review.
ENGCOM-8801 has been created to process this Pull Request

@engcom-Bravo engcom-Bravo moved this from Ready for Testing to Testing in Progress in High Priority Pull Requests Dashboard Feb 23, 2021
@engcom-Bravo
Copy link
Contributor

✔️ QA Passed

The result is the same as in the report above #32082 (comment)

@engcom-Bravo engcom-Bravo moved this from Testing in Progress to Merge in Progress in High Priority Pull Requests Dashboard Feb 23, 2021
@m2-community-project m2-community-project bot moved this from Merge in Progress to Ready for Testing in High Priority Pull Requests Dashboard Feb 23, 2021
@engcom-Charlie engcom-Charlie moved this from Ready for Testing to Merge in Progress in High Priority Pull Requests Dashboard Feb 23, 2021
magento-engcom-team pushed a commit that referenced this pull request Mar 7, 2021
@gabrieldagama
ENGCOM-8801: Deleting disabled category forces category flat and cata…
0bd9d67 
…log search re-indexed fully #32082
@magento-engcom-team magento-engcom-team merged commit c51447c into magento:2.4-develop Mar 7, 2021
@m2-assistant
Copy link

m2-assistant bot commented Mar 7, 2021

Hi @engcom-Charlie, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

@gabrieldagama gabrieldagama moved this from Merge in Progress to Recently Merged in High Priority Pull Requests Dashboard Mar 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aleron75 aleron75 aleron75 approved these changes

@danielrussob danielrussob danielrussob approved these changes

Assignees

@aleron75 aleron75

@danielrussob danielrussob

@engcom-Bravo engcom-Bravo

Labels
Auto-Tests: Covered All changes in Pull Request is covered by auto-tests Component: Catalog Component: CatalogSearch Priority: P2 A defect with this priority could have functionality issues which are not to expectations. Progress: accept QA: Added to Regression Scope Scenario was analysed and added to Regression Testing Scope Release Line: 2.4 Severity: S2 Major restrictions or short-term circumventions are required until a fix is available. Type: Feature/Improvement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Deleting disabled category forces category flat and catalog search re-indexed fully
7 participants
@engcom-Charlie @danielrussob @magento-engcom-team @aleron75 @engcom-Bravo @gabrieldagama @konarshankar07