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

Use the new order rejection reason when rejecting entire order #916

Closed
1 of 2 tasks
ymaheshwari1 opened this issue Jan 28, 2025 · 1 comment
Closed
1 of 2 tasks

Use the new order rejection reason when rejecting entire order #916

ymaheshwari1 opened this issue Jan 28, 2025 · 1 comment
Assignees
@ymaheshwari1
Labels
bug Something isn't working

Comments

@ymaheshwari1
Copy link
Contributor

Current behavior

Currently we are using REJECT_ENTIRE_ORDER as reason when rejection entire order, but we have defined a new reason REJ_AVOID_ORD_SPLIT for this case, thus need to update the same.

Expected behavior

Steps to reproduce the issue

Can you handle fixing this bug by yourself?

  • YES
  • NO

Environment details

  • Browser:
  • OS:
  • Code Version:

Additional information
@ymaheshwari1 ymaheshwari1 added the bug Something isn't working label Jan 28, 2025
@ymaheshwari1 ymaheshwari1 self-assigned this Jan 28, 2025
@ymaheshwari1 ymaheshwari1 changed the title Use the new order rejection reason when rejection entire order Use the new order rejection reason when rejecting entire order Jan 28, 2025
@ymaheshwari1 ymaheshwari1 moved this from 📋 Backlog to 🏗 In progress in Digital Experience Platform Jan 28, 2025
ymaheshwari1 added a commit to ymaheshwari1/fulfillment that referenced this issue Jan 28, 2025
@ymaheshwari1
Improved: logic to use the new rejection reason when rejecting entire…
cfa24b3 
… order(hotwax#916)
@ymaheshwari1 ymaheshwari1 mentioned this issue Jan 28, 2025
Merged
1 task
ymaheshwari1 added a commit that referenced this issue Jan 28, 2025
@ymaheshwari1
Merge pull request #918 from ymaheshwari1/#916
b0afd9b 
Improved: logic to use the new rejection reason when rejecting entire order(#916)
@ymaheshwari1 ymaheshwari1 moved this from 🏗 In progress to ⏸️ Dev complete in Digital Experience Platform Jan 28, 2025
@dt2patel
Copy link
Contributor

dt2patel commented Feb 4, 2025

No testing needed, assume data is in all systems, closing app side ticket

@dt2patel dt2patel closed this as completed Feb 4, 2025
@github-project-automation github-project-automation bot moved this from ⏸️ Dev complete to ✅ Done in Digital Experience Platform Feb 4, 2025
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
Development

No branches or pull requests
2 participants
@ymaheshwari1 @dt2patel