-
Notifications
You must be signed in to change notification settings - Fork 4
Enhancement for guest orders, products, orders and purchase items #3
base: master
Are you sure you want to change the base?
Conversation
…or each customer.
$customer_count = $opts[0]; | ||
$product_count = intval($opts['products']); | ||
$guest_order_count = intval($opts['guestorders']); | ||
$customer_count = intval($opts['customers']); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happens here if you pass in --customers 50000 3 5
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have made a fix to this to pick up items from the existing database. I'll submit a pull request once I have made the other changes.
Thanks for this! I've wanted to add this for ages. |
…of createFakeOrder() solely on createFakeProduct()
…ld be adjustable though.
I have a new patch that fixes some of these issues and I'll submit it pending your suggestions/feedback. Thanks heaps. |
…n. Made other changes as well based on feedback.
… have an existing item in the catalog. This means you should be able to create 'simple' product type items on the fly with an empty catalog.
I've added changes based on your review. Not sure if you want me to create a new branch and submit a new pull request or happy to use the 'new' commits on herbyme/magefaker? |
Hi guys, any news about this PR? |
For #1 and #2
Added options for guest orders, products, as well as orders for each fake customer. Didn't break it out into separate classes or separate commands (wanted to keep it as one command).