Magento-SMTP-Pro-Email-Extension icon indicating copy to clipboard operation
Magento-SMTP-Pro-Email-Extension copied to clipboard

Add support for SendGrid API

Open ravinggenius opened this issue 5 years ago • 10 comments

We need SendGrid API support at work, so I added it.

ravinggenius avatar Nov 13 '19 16:11 ravinggenius

Could this be looked into? As this will begin to be required for SendGrid users as of October 31st, 2020. Thanks

@aschroder

soupli avatar Oct 16 '20 12:10 soupli

Also need this feature, somebody did a fix on that ?

devaliere avatar Nov 20 '20 09:11 devaliere

Also need this feature, somebody did a fix on that ?

You could simply use the branch that @ravinggenius pushed

ravinggenius:support-sendgrid-api

I do not think this will ever be merged, cause there is no support for this module anymore

soupli avatar Nov 20 '20 09:11 soupli

Thank you @soupli. I add all the file, just don't know how to install (and where) the sendgrid API. Any idea, advice ?

devaliere avatar Dec 07 '20 12:12 devaliere

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/ SEE https://marekkalnik.tumblr.com/post/22929686367/composer-installing-package-from-local-git

ravinggenius avatar Dec 07 '20 16:12 ravinggenius

Hello @devaliere have you found where to integrate the sendgrid lib?

gcharpentier591 avatar Jan 04 '21 13:01 gcharpentier591

@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 :()

devaliere avatar Feb 18 '21 16:02 devaliere

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 avatar May 12 '21 22:05 earwickerh

@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.

ravinggenius avatar May 12 '21 23:05 ravinggenius

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:

{
    "repositories": [
        {"type": "composer", "url": "https://github.repo.repman.io"}
    ]
}

and then use composer require aschroder/smtp_pro

make sure you first remove any local installed packages under app/etc/local

ProxiBlue avatar May 22 '21 03:05 ProxiBlue