Jorge A. Torres
Jorge A. Torres
> What do you think about creating two more tests which inherits WC_REST_Orders_Controller_Tests and WC_REST_Order_Refunds_Controller_Test, then in the setup modifies the data store in use? That's a good idea. I've...
> would you mind confirming if it ok for us to merge this? (just want to make sure we are following correct etiquette 😄 ) Hi @jconroy! 👋 Now that...
Hi @smshmb! Thanks for reaching out and for providing details on how to reproduce the problem. We were able to reproduce the issue with version 2 of the REST API...
Thanks everyone for the report and apologize for the delay in responding. We were able to reproduce and will address in a future version.
Just mentioning that I've marked #33787 as ready for review, as the remaining changes were all in the `OrdersTableQuery` class. Hopefully that'll be merged soon and we can switch the...
Hi @alexmigf! Thanks for your contribution. We understand it's been a while since you last worked on this, so please let us know if you're still interested in seeing this...
Hi @alexmigf! 👋 Thanks for getting back to me. We discussed this a bit more on the team, and considering the `LIKE` implementation works everywhere we feel it shouldn't be...
Re-assigning to @barryhughes per p1659719396500269-slack-C0E1AV8T0.
Hi @barryhughes! Thanks for the review! I've added the docblock per your request. As for the async actions, I totally get what you mean. For those, "past due" means something...
> I think we could ship this as it is (and we can make further enhancements in the future if feedback suggests it is needed). wdyt? Sounds good to me....