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

[Order] Include order unit promotion adjustments and order item promotion adjustments in order promotion total #10846

Merged
merged 3 commits into from
Nov 12, 2019

Conversation

Tomanhez
Copy link
Contributor

@Tomanhez Tomanhez commented Nov 12, 2019

Q A
Branch? 1.5
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
License MIT

Fix for: #10800

@pamil pamil merged commit 1907f90 into Sylius:1.5 Nov 12, 2019
@pamil
Copy link
Contributor

pamil commented Nov 12, 2019

Thank you, Tomasz! 🥇

pamil added a commit that referenced this pull request Nov 14, 2019
…ons on 1.6 after upmerge (GSadee)

This PR was merged into the 1.6 branch.

Discussion
----------

| Q               | A
| --------------- | -----
| Branch?         | 1.6
| Bug fix?        | yes
| New feature?    | no
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | after #10846
| License         | MIT


Commits
-------

48e23f2 [Behat][Admin][Order] Fix scenarios for displaying promotions on 1.6 after upmerge
pamil added a commit that referenced this pull request Nov 14, 2019
…ons on master after upmerge (GSadee)

This PR was merged into the 1.6-dev branch.

Discussion
----------

| Q               | A
| --------------- | -----
| Branch?         | master
| Bug fix?        | yes
| New feature?    | no
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | after #10846, associated with #10853 
| License         | MIT


Commits
-------

a14a599 [Behat][Admin][Order] Fix scenarios for displaying promotions on master after upmerge
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.

None yet

2 participants