Skip to content

fix table name for UpdateLegacyStockStatus.php #3395

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
Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

fix table name for UpdateLegacyStockStatus.php #3395

wants to merge 3 commits into from

Conversation

duhon
Copy link
Contributor

@duhon duhon commented Aug 21, 2023 •
edited by m2-assistant bot
Loading

Description (*)

Fixed Issues (if relevant)

  1. Fixes magento/inventory#<issue_number>: Issue title

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)

Resolved issues:

  1. resolves [Issue] fix table name for UpdateLegacyStockStatus.php #3440: fix table name for UpdateLegacyStockStatus.php

@engcom-Hotel
Copy link
Collaborator

@magento run all tests

@engcom-Hotel engcom-Hotel added the Triage: Dev.Experience Issue related to Developer Experience and needs help with Triage to Confirm or Reject it label Sep 24, 2025
@engcom-Hotel engcom-Hotel moved this to Review in Progress in Inventory - Pull Request Progress Sep 24, 2025
@engcom-Hotel engcom-Hotel self-requested a review September 24, 2025 10:36
engcom-Hotel
engcom-Hotel approved these changes Sep 25, 2025
View reviewed changes
Copy link
Collaborator

@engcom-Hotel engcom-Hotel left a comment

Choose a reason for hiding this comment

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

Approving this PR for further processing.

@magento run Functional Test EE

@engcom-Hotel
Copy link
Collaborator

@magento run Functional Test EE

@magento-automated-testing Magento Automated Testing
Copy link

Failed to run the builds. Please try to re-run them later.

@engcom-Hotel engcom-Hotel moved this from Review in Progress to Reviewer Approved in Inventory - Pull Request Progress Sep 25, 2025
@engcom-Hotel
Copy link
Collaborator

@magento run Functional Test EE

@magento-automated-testing Magento Automated Testing
Copy link

Failed to run the builds. Please try to re-run them later.

@engcom-Hotel
Copy link
Collaborator

@magento create issue

@m2-assistant m2-assistant bot mentioned this pull request Oct 15, 2025
Open
4 tasks
@engcom-Hotel
Copy link
Collaborator

@magento run all tests

@engcom-Hotel
Copy link
Collaborator

✔️ QA Passed

Preconditions:

  • Magento 2.4.x installed and configured
  • PHP 8.4+
  • MySQL/MariaDB database
  • Multi-Source Inventory (MSI) modules enabled

Actual Result: ✔️

  • The code executes without errors
  • Table name is resolved correctly: 'cataloginventory_stock_status'
  • Stock status updates complete successfully

Before: ✖️
image

After: ✔️
The code resolves the table name successfully.

Some functional tests are still failing, hence checking further on this.

@engcom-Hotel engcom-Hotel moved this from Reviewer Approved to Acceptance Testing in Inventory - Pull Request Progress Oct 15, 2025
@engcom-Hotel
Copy link
Collaborator

@magento run Functional Tests B2B, Functional Tests EE

@engcom-Hotel
Copy link
Collaborator

The failed tests are known issues and does not belongs to this PR:

MC-27571: https://jira.corp.adobe.com/browse/ACQE-8682
MC-32333: https://jira.corp.adobe.com/browse/ACQE-8511

Please refer to the below screenshots:

image image

Hence moving further with this PR.

Thank you

@engcom-Hotel engcom-Hotel moved this from Acceptance Testing to Ready for Delivery in Inventory - Pull Request Progress Oct 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@engcom-Hotel engcom-Hotel engcom-Hotel approved these changes

Assignees

No one assigned

Labels

Progress: accept Triage: Dev.Experience Issue related to Developer Experience and needs help with Triage to Confirm or Reject it

Projects

Inventory - Pull Request Progress
Status: Ready for Delivery

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Issue] fix table name for UpdateLegacyStockStatus.php

2 participants

@duhon @engcom-Hotel