Adil Basri
Adil Basri
Now that I think about it we should add every virtual fields that exists as a column, should we just throw a bunch of keys inside ? or is there...
> Thanks for the contribution. Definitely makes sense to upgrade. > > But just to be clear and in case anyone sees this PR, the version we currently depend on...
> This looks good to me, do you have a video or screen shot of the result? Yes here it is, but there is two issues I discovered while testing...
Ok fixed the filter issue that was setting `undefined` instead of `o_total` For the orders that were returned with the not correct amount, the issue is coming from my dataset...
@adrien2p here is the demo https://github.com/user-attachments/assets/2998bd10-aa9c-42db-a5ab-d72fa8a7594e
> thanks man 🎉 should we add a new integration tests that filters with summary total on the order? hey @adrien2p , I addressed the feedacks, is it ok to...
> > > thanks man 🎉 should we add a new integration tests that filters with summary total on the order? > > > > > > hey @adrien2p ,...
https://github.com/medusajs/medusa/pull/14191