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.

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

Configurable stock status is not correct when using custom stock (MSI) #3394

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

Kannakiraj123
Copy link

@Kannakiraj123 Kannakiraj123 commented Aug 3, 2023

Description (*)

This PR fix when MSI enabled to product ,the configurable product status must be updated when a child's product status is changed.

Fixed Issues (if relevant)

  1. Fixes #36154

magento/magento2#36154

Manual testing scenarios (*)

  • create new source in admin > store > sources, i.e : source A
  • create new stock in admin > store > stock, i.e : custom stock
  • assign source A to Custom stock, and also assign custom stock to main website
  • create new configurable product, i.e : test product
    - add simple product to the configurable product in configuration tab
    - assign simple product to source A
    - fill quantity, i.e : 1
  • and save new created configurable product. (in this step the configurable product stock = In Stock)
  • now change the simple product :
    - Source Item Status : from In Stock to Out Of Stock
    - Qty : from 1 to 0
    - save simple product
  • check configurable product (the stock status is still In Stock, it must be Out Of Stock)

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)

…ck-status-is-not-correct-when-using-custom-stock-msi
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.

2 participants