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

DEVDOCS-6175: [update] update order of operations #721

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

Conversation

bc-traciporter
Copy link
Collaborator

@bc-traciporter bc-traciporter commented Dec 20, 2024

DEVDOCS-6175

What changed?

Moved customer group discounts and bulk pricing after Product modifier in the table.

Release notes draft

Bug fix: Moved customer group discounts because they are shown before modifier rules when they apply after, as does bulk pricing.

Anything else?

ping {names}

@bc-traciporter bc-traciporter added the ready for review This PR is complete and awaiting a review. label Dec 20, 2024
Copy link
Contributor

@megdesko megdesko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bc-traciporter what do you think of splitting into two versions: one for price lists and one for non price lists requests? Since you can't use both features at the same time, this may make more sense to users.

@bc-traciporter
Copy link
Collaborator Author

@megdesko are you thinking of something like this?
Screenshot 2024-12-23 at 11 31 30 AM
Screenshot 2024-12-23 at 11 35 43 AM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review This PR is complete and awaiting a review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants