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

Fix the category tree depth calculation bug #102

Conversation

dmytro-ch
Copy link
Contributor

Description

Fixed the issue with incorrectly calculated maximum level.

Fixed Issues

  1. Category tree depth calculation issue #100: Category tree depth calculation issue

Manual testing scenarios

The result response should contain the last child item (specified on the deepest level according to the request).

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)

@dmytro-ch dmytro-ch added bug Something isn't working coverage small labels Jun 30, 2018
@dmytro-ch dmytro-ch self-assigned this Jun 30, 2018
@dmytro-ch dmytro-ch requested a review from paliarush June 30, 2018 11:05
@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Jun 30, 2018

CLA assistant check
All committers have signed the CLA.

@paliarush
Copy link
Contributor

Resolves #21

@magento-engcom-team magento-engcom-team merged commit 9116d82 into 2.3-develop Jul 19, 2018
@naydav naydav deleted the graphql-fix-the-category-tree-depth-calculation-issue branch April 12, 2019 19:32
magento-engcom-team pushed a commit that referenced this pull request May 31, 2019
magento-engcom-team added a commit that referenced this pull request Jun 6, 2019
…tore_id from … #102

 - Merge Pull Request magento-commerce/async-import#102 from comwrap/async-import:2.3-develop-bulk-api-multistore-hot-fix
 - Merged commits:
   1. fd6cf5e
   2. 80732c8
   3. 51ac4d8
   4. 331e311
   5. dad12c6
   6. 81ecd90
   7. ab2f000
   8. 48104dc
   9. 460a249
   10. c336e36
   11. 038615b
   12. fb06c33
   13. 06bc3a2
magento-engcom-team pushed a commit that referenced this pull request Oct 12, 2019
Merging MC-20181 changes to MC-20644 branch
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.

5 participants