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

Improved: Enhanced ATP rule group execution logic to make it more efficient #39

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ravilodhi
Copy link
Contributor

@ravilodhi ravilodhi commented Dec 13, 2024

Enhancements Made:

Introduced a Parameter:
Added a parameter named scanAllProducts to control whether to scan all products or only the updated products.

Scan All Products Conditions:
Rule Group Modified: If the rule group has been modified since the last successful run, process all products.
Rule Group Never Run: If the rule group has never been run before, process all products.

Conditional Scanning Based on Facilities and Facility Groups:
If no rule group has been updated since the last successful run:

1) Threshold Rules:
Check if rules are configured to consider all configuration facilities.

2) Safety Stock, Shipping, and Store Pickup Rules:
Check if rules are configured to consider all facility groups. If such rules exist, verify whether any new facility has been added to the product store since the last run. If so, scan all products.

3) Selected Facility Groups (for Safety Stock, Shipping, and Store Pickup):
Even if no rule is configured for all facility groups, check the facility group members of the selected groups. If any new member has been added since the last successful run, scan all products.

Updated Products:
For all remaining cases, fetch and process only the products updated after the last successful run.

…rule group is modified since the last run; otherwise, process only products updated since the last successful run (#38).
@ravilodhi ravilodhi self-assigned this Dec 16, 2024
@ravilodhi ravilodhi changed the title Improved: Enhanced ATP rule group logic: process all products if the rule group is modified since the last run; otherwise, process only products updated since the last successful run (#38). Improved: Enhanced ATP rule group execution logic to make it more efficient Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant