- Issue created by @jsacksick
- Status changed to Needs review
10 months ago 3:19pm 25 January 2024 - last update
10 months ago CI error - 🇳🇿New Zealand sjmcl
That line of code has cost me several hours trying to fix a bug that looked like it was caused by the Currency Resolver module. Removing that line fixed it.
Thanks for posting that patch.
- 🇮🇱Israel jsacksick
I've never committed this due to the tests not completing with this change for whatever reason...
- Merge request !302Issue #3416998 by jsacksick: Stop unsetting the order ID reference from the order refresh. → (Open) created by jsacksick
- 🇮🇱Israel jsacksick
A lot of tests are failing with this change lol... The patch has been applied to one of the projects I'm working on for months without side effects (at least not that I'm aware of).
- 🇨🇭Switzerland mazze
Thank you so much for the patch. I already started thinking about calling the customer, telling him that we have a major problem, now everything is working properly:-)
- 🇮🇱Israel jsacksick
I wanted to commit the fix, but the tests are not completing without it... So this line is indeed fixing something... But breaking others...