Order Quantity Changed or Order Changed event

OK. I’ve rolled back Ticket Total Changed execution for canceled orders as it have it’s own event. While doing that I’ve also noticed Order Canceled event does not work fine as it executes before order gets removed from ticket. As event name suggests I’ve changed it to work after order removed from ticket. Now it seems working fine. After next release please let me know if we’ll still need ticket amount changed event for canceled orders. @QMcKay thank you very much for helping me on improving it and @pipo sorry for the hijack.

1 Like

Cancelling an Order should trigger Ticket Total Changed event, imo, because, well, the total is changed, so yes.

Not sure what you mean by banner. If you are suggesting that firing the Pole Items Print Job on Order Cancelled Event will work in this case, that’s good so I can make this work now, but I still think it should trigger Ticket Total Changed Event as well.

I just tested this with Order Cancelled Event, and it does not work correctly…

1 Tea        : 40.00
1 Water Large: 50.00
1 Hot Chocola: 60.00
       Total: 150.00

Cancel Tea:

1 Tea        : 40.00
1 Water Large: 50.00
1 Hot Chocola: 60.00
       Total: 150.00

Strangely enough, the Display is being sent an update (I can see it via my text editor change detection), but the text is identical.

1 Like

LOL sorry for multiple posts but I thought I should also execute Ticket Total Changed event as ticket total changes after we cancel orders. So I’ve tweaked it to execute Order Canceled event after Ticket Total Changed event. I hope that won’t change anything on existing configurations but it should work that way :slight_smile:

2 Likes

I mean with “banner” displaying something on pole display as “XXX Canceled”. I think that will be possible with new implementation.

1 Like

Excellent, that’s the way I think it should work too. I’ll report back after the update.

I am testing this and this messed up all my configuration!!!

LOL!!! :wink: just kidding… hope you dong get mad…

G.

1 Like

Pole Display working as it should in all scenarios - thanks @emre!

2 Likes