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

Fixed: wrong excluded atp calcuation and updated the job card title on details page(#240) #241

Merged
merged 4 commits into from
Oct 9, 2023

Conversation

ymaheshwari1
Copy link
Contributor

Related Issues

Closes #240

Short Description and Why It's Useful

  • Updated excluded atp calculation to use atp and onlineAtp instead of using qoh
  • Improved job card title from Category and Brokering jobs to Related jobs as in some case we might not get brokering/category jobs and in that case the card title becomes irrelevant

Screenshots of Visual Changes before/after (If There Are Any)

Before:

image

After:

image

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change

Contribution and Currently Important Rules Acceptance

@ravilodhi ravilodhi merged commit e688d95 into hotwax:main Oct 9, 2023
1 check passed
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.

Exclude Reserved Sales Order Count from ATP Computation in Online ATP Calculation Card
2 participants