- Status changed to Needs work
over 1 year ago 4:55pm 1 March 2023 - 🇨🇭Switzerland Lukas von Blarer
The patch in #11 works for me as well.
But I have one remaining issue. When using this for a promotion that gives a discount on the shipping fee, the promotion is only applied after saving the order. Hence when continuing in the checkout flow to the next step. This is a problem when using AJAX to recalculate shipping costs. The order totals fetched via AJAX don't apply the promotion correctly, but after clicking the "Continue" button, the promition is applied correctly.
- 🇺🇸United States rszrama
After reviewing this with jsacksick today in the context of a client using the patch in #7. I actually can't remember why I thought it would be better for us to have a separate condition as zaporylie implemented for us in #11. The issue I have with it in hindsight is that it's now ambiguous - which one should I be using? Is the regular "Order total" condition somehow not a current order total?
Additionally, I think I may have introduced some scope creep in responding to bojanz re: how to accommodate different adjustment types. Sufficient for the day is its own trouble ... we could have just landed a patch that differentiated between the current order total vs. order item subtotal a while back and figured out how to further subdivide in a follow-up issue.
I propose we go back to jsacksick's patch for now with some slight adjustment to the labels:
Total to compare against
- Order total
- Sum of order item totals
The point of the description is to try to make it clear what "current" means ... the problem is these conditions are used in multiple contexts, and "current" isn't terribly descriptive. Then we can create a follow-up issue to add a third option that would let us get to some sort of price + adjustments selection widget.
- Status changed to Needs review
over 1 year ago 12:53pm 4 April 2023 - 🇮🇱Israel jsacksick
One thing that was forgotten including in the patch from #7 is the schema update (commerce_order.schema.yml) we need to add the new setting there.
-
jsacksick →
committed 98bbaed6 on 8.x-2.x authored by
tBKoT →
Issue #2938729 by tBKoT, krystalcode, jsacksick, matteodem, zaporylie,...
-
jsacksick →
committed 98bbaed6 on 8.x-2.x authored by
tBKoT →
- Status changed to Fixed
over 1 year ago 11:49am 7 April 2023 -
jsacksick →
committed 5712cd29 on 3.0.x authored by
tBKoT →
Issue #2938729 by tBKoT, krystalcode, jsacksick, matteodem, zaporylie,...
-
jsacksick →
committed 5712cd29 on 3.0.x authored by
tBKoT →
Automatically closed - issue fixed for 2 weeks with no activity.