Skip to content

Low stock report is showing empty grid #9408 #14705

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.

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

Closed
wants to merge 1 commit into from

Conversation

EujenMerezhka
Copy link

@EujenMerezhka EujenMerezhka commented Apr 14, 2018

Description

Fixed Report Low Stock Products

Fixed Issues (if relevant)

  1. Low stock report is showing empty grid in magento 2.1.6 #9408: Low stock report is showing empty grid

Manual testing scenarios

  1. Login to admin panel
  2. Create product with qty as 0 (Product >> Catalog >> Create new Product)
  3. Check the low stock report ("Report >> Products >> Low stock)
    Expected result:
    It will show the grid with value.

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)

@sidolov sidolov changed the base branch from Fixed-9408 to 2.2-develop April 14, 2018 15:04
@magento-engcom-team magento-engcom-team added this to the April 2018 milestone Apr 14, 2018
@sidolov
Copy link
Contributor

sidolov commented Apr 23, 2018

Hi @EujenMerezhka , during QA activities we found that we can't reproduce the issue on 2.2-develop, but it exists on 2.2.3 version. Looks like we fixed it in the scope of another issue. Can you verify steps on 2.2-develop? Thanks!

@sidolov
Copy link
Contributor

sidolov commented May 9, 2018

@EujenMerezhka , I am closing this PR now due to inactivity.
Please reopen and update if you wish to continue.
Thank you for collaboration!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants