search

coinbase / coinbase-commerce-woocommerce

Accept Bitcoin on your WooCommerce-powered website.
https://commerce.coinbase.com/integrate/woocommerce
Apache License 2.0
58 stars 40 forks source link

No "New Order #XXX" sent to site admin #13

Closed briswift closed 5 years ago

briswift commented 5 years ago

This may be tied to issue #7, but I haven't received a notification from my website that I've received a new order. I've confirmed that I'm receiving this email with other payment methods.

Thanks,

ciscospirit commented 5 years ago

Hello...

i download the latest changes from github and installed the plugin with this changes and the "processing" status update works now BUT it will not trigger the mail for "new order" for the site admin or the "processing order" for the customer...

is there also a fix avilable somehwere?

ciscospirit commented 5 years ago

@briswift do you have maybe a solution for this?

briswift commented 5 years ago

@briswift do you have maybe a solution for this?

I Haven't found a solution though I must admit I haven't done much searching. Thanks for the link and debug log note. Mostly wanted to raise this so the plugin devs were aware of it.

ciscospirit commented 5 years ago

how you deal in the meantime with that? or did you change to another plugin? I can live without the site admin mails but i need the "processing order" mails to the customers... :(

sadly i am too bad in coding to fix this in the code by my self :/

ciscospirit commented 5 years ago

short update: i tried it now with payment method "cash on delivery" and if i choose this, the processing mail got sent instandly.... so it is definitely a problem with the coinbase commerce plugin, that this not trigger the mail send process.

ciscospirit commented 5 years ago

Maybe here is the answer? https://stackoverflow.com/questions/36178717/how-to-trigger-wordpress-woocommerce-email-on-payment-gateway-plugin

https://docs.woocommerce.com/wc-apidocs/class-WC_Email_Customer_Processing_Order.html

ciscospirit commented 5 years ago

@guacamoli or @imeleshko do you have maybe an idea how to fix the "no processing mail" problem?

briswift commented 5 years ago

how you deal in the meantime with that? or did you change to another plugin? I can live without the site admin mails but i need the "processing order" mails to the customers... :(

sadly i am too bad in coding to fix this in the code by my self :/

Not exactly an elegant solution, but I check my website each day. If I see an an order in the "Blockchain Pending," I check my Coinbase Commerce account to ensure payment was received then switch back to Woocommerce and change status from "Blockchain Pending" to "Processing". Doing this manually triggers the processing email to the customer (but no new order received email to me). Obviously this is not ideal but it works for me for the time being. Try a test transaction on your site and see if this will work for you?

guacamoli commented 5 years ago

Hey y’all, sorry but I won’t have any bandwidth to make updates for a while. Any contributions are welcome if anyone wants to open a PR 🙏🏽

ciscospirit commented 5 years ago

@briswift the "Blockchain Pending" Problem is solved if you use the latest fixed files from this GIT and replace the files from your wordpress plugin installation.

@guacamoli ah thats not good :/ cause i can't solve this "WC_Email_Customer_Processing_Order" problem on my own... :/

briswift commented 5 years ago

@briswift the "Blockchain Pending" Problem is solved if you use the latest fixed files from this GIT and replace the files from your wordpress plugin installation.

@guacamoli ah thats not good :/ cause i can't solve this "WC_Email_Customer_Processing_Order" problem on my own... :/

Thanks! Updated the files and that works for me. So long as I continue getting the "payment completed" emails from the Coinbase, I can just manually send out the order processing emails from woocommerce.

sahil-cb commented 5 years ago

Closing this since it's resolved.

equiss commented 5 years ago

Hello,

This issue should not be closed. WC is not sending order emails for orders paid via this plugin.

This is due to using the "blockchainpending" status. When the status change from "blockchainpending" to "processing" occurs, the notification sent out is not hooked into any email action.

The fix here is simple... 2 options. 1.) (Kills "blockchainpending") Quick and dirty: Change https://github.com/coinbase/coinbase-commerce-woocommerce/blob/master/class-wc-gateway-coinbase.php#L366 "blockchainpending" to "pending".
2.) (keeps "blockchainpending") Add action hooks to the WC emails for woocommerce_order_status_blockchainpending_to_processing_notification (see: https://github.com/woocommerce/woocommerce/blob/07351d5e11f8d5dfc3620577ec5fe3128e704e81/includes/emails/class-wc-email-new-order.php#L42 )