magepal / magento2-gmail-smtp-app

Configure Magento 2 to send email using Google App, Gmail, Amazon Simple Email Service (SES), Microsoft Office365 and many other SMTP (Simple Mail Transfer Protocol) servers
https://www.magepal.com
319 stars 147 forks source link

compatible with magneto 2.2.5? no order confirmation mail, no shipment mail #117

Closed Jacob-Wilsons closed 6 years ago

Jacob-Wilsons commented 6 years ago

Is this extension compatible with magneto 2.2.5?

I have the same issue as described issues/16231

Preconditions magento 2.2.5 magento2-gmailsmtpapp

Steps to reproduce

Upgrade magento to 2.2.5 Check Stores-> Advanced -> System -> Gmail/Google Apps SMTP Pro, all configurations are ok. Click "Send test email", can receive test mail correctly. Can send mail when a new user is registered. Can send mail when password reset. Can send mail when newsletter subscribed/ unsubscribe

After user placing an order (payment via paypal, or payment as "cash / check" method), no confirmation mail, after ship the order, no shipment mail.

I'm not sure, if this issue is related to magento2-gmailsmtpapp extension or to magento core file ?

Thanks

srenon commented 6 years ago

This is a core bug... take a look at https://github.com/magepal/magento2-gmail-smtp-app/issues/116 and https://github.com/magepal/magento2-gmail-smtp-app/issues/113

Jacob-Wilsons commented 6 years ago

@srenon tank you. I found the bug and fixed it in TransportPlugin.php

I have a question about updating gmail-smtp-app module. If installed via composer it is all clear. What is the correct way to update it if was installed manually without using composer? Thanks

srenon commented 6 years ago

You could either:

  1. Delete folder app/code/MagePal and install using composer (all your setting should still be save in the db)

  2. Download master branch at https://github.com/magepal/magento2-gmail-smtp-app/archive/master.zip and replace the content of app/code/MagePal

Jacob-Wilsons commented 6 years ago

Method above worked just fine. Thank you @srenon