-
Notifications
You must be signed in to change notification settings - Fork 209
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for SendGrid API #274
base: master
Are you sure you want to change the base?
Conversation
Could this be looked into? As this will begin to be required for SendGrid users as of October 31st, 2020. Thanks |
Also need this feature, somebody did a fix on that ? |
You could simply use the branch that @ravinggenius pushed
I do not think this will ever be merged, cause there is no support for this module anymore |
Thank you @soupli. |
You should be able to install my fork if you have Composer. Something like (not tested): composer config repositories.repo-name vcs https://github.com/ravinggenius/Magento-SMTP-Pro-Email-Extension.git
composer require aschroder/smtp_pro:dev-support-sendgrid-api --prefer-source SEE https://webkul.com/blog/install-packages-using-composer-local-git-repo/ |
Hello @devaliere have you found where to integrate the sendgrid lib? |
@gcharpentier591 : I am not able to make it working. I install it (manually through git clone and then composer install) and then choose Sengrid APi (and copy my API) but the test mail is not working so I am block on this issue (unfortunately :() |
Thanks @ravinggenius! The self-test doesn't run when using the SendGrid API option. Also noticed there's only a field for API key (no "User" or "API key ID field") since self-test wasn't running, not sure if it's working. Just realized @devaliere had the same issue. |
@earwickerh it worked for me when I wrote it originally. Please feel free to add to what I have here or even open a new merge request. I no longer work for the company I did when I opened this merge request, and I have no need for Magento, Sendgrid or even PHP in my current role. I'm not trying to leave anyone hanging, but I cannot offer any support beyond what I have already done in this merge request. |
worked fine for me, thank you for this effort @ravinggenius FYI, I published a new version/release on version 2.1.0 on a fork, and placed it in on repman.io for composer install composer config --global --auth http-basic.github.repo.repman.io token ec42f7dd9269f0d9355f94279d221d3f07c4f8c70c9344a1ef27ddbc0a07a8d6 Add these lines to your composer.json file, or add a new repository URL if you already have one or more:
and then use make sure you first remove any local installed packages under app/etc/local |
We need SendGrid API support at work, so I added it.