The latest release of the SendGrid API package includes an change in the class name for the main client. This causes the module to fail in D7 on a fresh install because the SendGrid class can't be found. Tested a patch that will be uploaded presently.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dustin LeBlanc created an issue. See original summary.

Perignon’s picture

I have not tested the newest API package with the module. Even though I write the API! If you found a problem, please let me know. It could require a fix on the API.

dustinleblanc’s picture

Hey Perignon,

Here is the patch, tested it still works, just a simple class name change :)

Perignon’s picture

Ah yes. the PSR4 changes. Thank you so much for the patch. I will get it committed in a few minutes.

Perignon’s picture

Status: Active » Fixed

D.O is slow picking up commits. But this is fixed! Thanks!

dustinleblanc’s picture

Hey Perignon,

Great! Are you rolling a release or should we instruct folks to pull dev?

Best Regards,
Dustin LeBlanc
Customer Success Engineer | Pantheon

Perignon’s picture

Rolling a release as I type this.

Perignon’s picture

Well I say I am doing that but D.O is not picking up git changes. I tagged a new release but cannot create a release package here.

We are breaking out of beta and going full release.

Perignon’s picture

Thumbs up on Travis CI build. Now waiting on D.O...

https://travis-ci.org/taz77/drupal-sendgrid_integration/builds/141729595

Perignon’s picture

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.