Corey McKrill

Oops, thanks for the heads up @om4james! There's an [extra step](https://developer.woo.com/docs/contributing-technical-documentation/#2-creating-new-pages) when new images are added that hasn't happened yet, but I'll follow up about it.

Probably need to rename, _and_ refactor to make use of query params instead of a list of IDs. But agreed that changing the private method is probably the best option.

@kkmuffme thanks for the feedback. Where in the docs do you think would be best to put a clarification like this?

@boonedev I suspect this is happening because the full Action Scheduler package is still getting loaded during the request that deletes it, so it still has a callback hooked to...

Marking this as "enhancement" in case we want to investigate whether there's anything protections against this that we could add within Action Scheduler itself...

@kkmuffme this sounds like a reasonable improvement. Happy to take a PR on this if you're so inclined!

@gin0115 it's a bit confusing why the Action Scheduler migration is trying to run, since that is from a pretty old version. Did you upgrade it recently? Can you tell...

Related: https://meta.trac.wordpress.org/ticket/2992

Another warning when generating orders after already generating customers and products: ``` Generating orders 20 % [=============================> ] 0:00 / 0:00Warning: Trying to access array offset on value of type...

@masteradhoc it's going to be a while before I have time for another round of improvements, but I agree that this should get reviewed soon.