emspay / deprecated-magento1

Deprecated - EMS Pay - Magento1 plugin
http://emspay.eu
BSD 2-Clause "Simplified" License
0 stars 1 forks source link

Bug: Transactional emails (order confirmation, invoice) not being sent automatically for paid orders. [EST = 1h] #2

Closed sereshqua closed 7 years ago

sereshqua commented 7 years ago

Steps to reproduce:

1) Go to test shop https://54.154.48.86/ 2) Register 3) Log in 4) Add item to cart 5) Proceed through checkout 6) Pay 7) Check email for order confirmation and invoice

Actual result: Transactional emails (order confirmation, invoice) are not being sent automatically for paid orders.

Expected result: Transactional emails (order confirmation, invoice) are being sent automatically for paid orders.

Attachment: https://www.dropbox.com/s/zx97pbnyfu0l9b4/Screen%20Shot%202017-09-21%20at%2013.24.16.png?dl=0

sereshqua commented 7 years ago

Now invoice is only can be sent manually from admin panel: Sales -> Invoices -> Open any invoice -> Click on send email button

dligthart commented 7 years ago

@sereshqua can you update the status of this issue? Has it been reproduced?

sereshqua commented 7 years ago

we are waiting for further info, some emails have been sent in order to get chhay [10:46 AM] It's not related to my Magento shop

[10:46] It's related to the shop of a merchant of ours

[10:46] I also don't know how the merchant has configured his Magento plugin. I think it's best if you would send an e-mail to the person who implements this:

ghost commented 7 years ago

Pull request ready with changes.

ghost commented 7 years ago

I've tested again after upgrade. For Orders - email was sent correctly. Invoice email was sent but with $0.00 values. I'm solving it.

dligthart commented 7 years ago

Ok thank you - please create a PR a soon as it's done.

On Fri, Oct 6, 2017 at 3:17 PM, witoldprucnel notifications@github.com wrote:

I've tested for Invoice as well. Now - Invoice email is sent but with $0.00 values. I'm digging into it to solve it.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/emspay/magento1/issues/2#issuecomment-334751761, or mute the thread https://github.com/notifications/unsubscribe-auth/AABkh_ikCnbGTdtx_iKIrIiWiqJ0v-Zuks5spihegaJpZM4PgXk4 .

--

Met vriendelijke groet,

Dave Ligthart

Chief Technology

DLWT

dave@dlwt.nl

www.dlwt.nl

+31(0)6-221-20-551


DLWT: A web technology consultancy with a focus on delivering turnkey full stack managed web development sourcing.


ghost commented 7 years ago

Done. I've put the updated code on test https://54.154.48.86. Order and Invoice emails are generated correctly. I also add multi invoice generation as well when more than one invoice for order is set. For each Invoice will send separate Invoice Email.

dligthart commented 7 years ago

@witoldprucnel merged the PR @sereshqua can you update the status

sereshqua commented 7 years ago

verified as fixed