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.

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

Exclude Reserved Sales Order Count from ATP Computation in Online ATP Calculation Card #240

Closed
2 tasks
Dhiraj1405 opened this issue Oct 9, 2023 · 0 comments · Fixed by #241
Closed
2 tasks

Exclude Reserved Sales Order Count from ATP Computation in Online ATP Calculation Card #240

Dhiraj1405 opened this issue Oct 9, 2023 · 0 comments · Fixed by #241
Assignees
@ymaheshwari1
Labels
bug Something isn't working

Comments

@Dhiraj1405
Copy link
Contributor

Current behavior

In the Online ATP calculation card, the Excluded ATP computation includes the reserved sales order count, which affects the accuracy of the Excluded ATP calculation.

Expected behavior

Excluded ATP computation does not include sales order reservation in the Online ATP calculation card.

Steps to reproduce the issue

Can you handle fixing this bug by yourself?

  • YES
  • NO

Environment details

  • Browser:
  • OS:
  • Code Version:

Additional information
@Dhiraj1405 Dhiraj1405 added the bug Something isn't working label Oct 9, 2023
ymaheshwari1 added a commit to ymaheshwari1/preorder that referenced this issue Oct 9, 2023
@ymaheshwari1
Fixed: wrong excluded atp on catalog details page by calculating excl…
b0d035f 
…uded atp using atp and onlineAtp(hotwax#240)
ymaheshwari1 added a commit to ymaheshwari1/preorder that referenced this issue Oct 9, 2023
@ymaheshwari1
Improved: jobs card title(hotwax#240)
d20e117
@ymaheshwari1 ymaheshwari1 mentioned this issue Oct 9, 2023
Merged
1 task
ymaheshwari1 added a commit to ymaheshwari1/preorder that referenced this issue Oct 9, 2023
@ymaheshwari1
Improved: condition to check atp type(hotwax#240)
678d288
ymaheshwari1 added a commit to ymaheshwari1/preorder that referenced this issue Oct 9, 2023
@ymaheshwari1
Improved: ordering of text in locale(hotwax#240)
17c01af
ravilodhi added a commit that referenced this issue Oct 9, 2023
@ravilodhi
Merge pull request #241 from ymaheshwari1/preorder/#240
e688d95 
Fixed: wrong excluded atp calcuation and updated the job card title on details page(#240)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ymaheshwari1 ymaheshwari1

Labels
bug Something isn't working
Projects
Digital Experience Platform
Archived in project
Milestone
No milestone
2 participants
@ymaheshwari1 @Dhiraj1405