Commerce Waitlist

I had some initial issues getting this setup but it seemed to be tied mostly to the Flags 3.x module. I found that if you run update.php immediately after installing Flags that seems to help there. Here are a few things I found

1. Should probably note that Commerce Stock 2 is required since that is still an alpha version and not the default one mosts people will grab.
2. In README.txt it would be helpful to see a brief description of how to use the module as well as the paths to any admin pages
3. I don't know that Waitlist needs it's own package, might be better to lump it into Commerce (Stock) or Commerce (Contrib)
4. The link that is created to add a product to the waitlist is coming out for me as "Add [commerce-product:title] to waitlist"
5. When I put a product back in stock and it presumably tries to send the email I get the following error.

EntityMetadataWrapperException: Unknown data property field_product_ref. in EntityStructureWrapper->getPropertyInfo() (line 339 of/Applications/MAMP/htdocs/kickstart/profiles/commerce_kickstart/modules/entity/includes/entity.wrapper.inc).

Fixing wrong reason for using flag-3.x

All of the issues I mentioned above appear to be resolved and everything appears to be working as expected.

One thing I would throw out there as something to consider going forward is the flexibility of the notifications. Right now if I mark a product on my waitlist and it comes back in stock I will receive the notification. If it then goes out of stock, and back in stock again I get no notification. It might be nice if it was a bit more flexible in that sort of a situation. Also, when I place and complete an order it would be cool if it could cross check my waitlist and if I'm buying an item from the list get a message somewhere that says "hey, you're buying this item from your list, do you want to remove it from the waitlist?" or something to that effect.

Hi Itamar,

Very nice module that is probably needed by many.
I found it to be well documented, short and consice.

Please see several minor issues to consider:
1. Adding details the README file such as installation guidance and Q&A
2. Implementing L10N support at commerce_waitlist_flag_default_flags + commerce_waitlist_default_message_type

Best Regards,
Moshe

Closing due to lack of activity. Feel free to reopen if you are still working on this application.

Updated issue summary.