Skip to content

Conversation

@swnsma
Copy link
Contributor

@swnsma swnsma commented Sep 30, 2020

Description (*)

Stop stock data load on after load event.
Inheritance of #3039

See #3202

Fixed Issues (if relevant)

  1. Load product salability data on-demand #3201

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)

@swnsma
Copy link
Contributor Author

swnsma commented Sep 30, 2020

@magento run all tests

@swnsma swnsma mentioned this pull request Sep 30, 2020
oleksandrkravchuk and others added 13 commits October 1, 2020 12:29
Apply code review changes.
Move bundle status check to plugin.
Move configurable plugin to around.
Both facts comes from fact for bundle and configurable produts, salability status is depends only on salability status of their children (and does not depend on the condition chain).
Add GraphQl support for products assignment intersection.
Incapsulate internal logic in single class and expose extension point for extension attributes.
Move changes from #3039.
Remove excess code and TODO's.
@novikor novikor force-pushed the load-product-salability-data-on-demand branch from 2705aad to 5818efe Compare October 1, 2020 09:32
@novikor
Copy link
Contributor

novikor commented Oct 1, 2020

@magento run Integration Tests

@novikor
Copy link
Contributor

novikor commented Oct 1, 2020

@magento run Static Tests

@novikor
Copy link
Contributor

novikor commented Oct 2, 2020

Let's close it in favor of #3208

@novikor novikor closed this Oct 2, 2020
@sidolov sidolov deleted the load-product-salability-data-on-demand branch October 2, 2020 16:11
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.

4 participants