Closed michi-dev closed 1 year ago
No problem, glad to see it worked @Luc45
@samueljseay can u check and merge this one also?
Hi @michi-dev,
Apologies not getting to this sooner. It looks like there are some conflicts now. Would you mind resolving those so that we can give this a re-review?
It looks like there are still some conflicts here @michi-dev just bumping this in case you missed it. Sorry again for the delayed response on my part.
@samueljseay @jonathansadowski should now be fine - can you check?
Hi @michi-dev,
Sorry for the delay in getting back to this. Looks great, thanks for your contribution!
All Submissions:
Changes proposed in this Pull Request:
Closes #70.
I have added more functionality than described in the issue #70. This needs a bit further explanation. I have divorced any shipping / billing address in three parts:
Every customer, who gets generated, has those three parts. Every part has a random generation. I added the feature that at the customer generation in Customer.php: possibility A: billing = shipping (was done before) possibility B: 1/2 billing != shipping (added with this pull request)
therefore, the shipping address can have different details for
every part has its own random generation. with all three parts coming together its a unique order with in total (225) 40 combinations and therefore test cases.
How to test the changes in this Pull Request:
1.use
wp wc generate customers 50
to directly test the customer generation 2.usewp wc generate orders 50
to see if the customers were correctly saved to the ordersOther information:
Changelog entry
FOR PR REVIEWER ONLY: