Skip to content
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

missing sendgrid-php.php #406

Closed
nzhome opened this issue Jun 23, 2017 · 6 comments
Closed

missing sendgrid-php.php #406

nzhome opened this issue Jun 23, 2017 · 6 comments

Comments

@nzhome
Copy link

nzhome commented Jun 23, 2017

Why does .gitignore in this project hide sendgrid-php.php. I spent 30 minutes looking high and low on the internet for this file. Where is it!?

@nzhome
Copy link
Author

nzhome commented Jun 23, 2017

seems similiar to issues 340 and 341
who is maintaining this zip file?
https://sendgrid-open-source.s3.amazonaws.com/sendgrid-php/sendgrid-php.zip
is it the latest and greatest?

@nzhome
Copy link
Author

nzhome commented Jun 23, 2017

where is the best sendgrid-php.php maintained? anyone got a link? and no I don't use composer because i am using hostgator.

@thinkingserious
Copy link
Contributor

Hello @nzhome,

It looks like the script we use to autogenerate that file is broken: https://github.com/sendgrid/sendgrid-php/blob/master/scripts/s3upload.sh#L10

We do have a fix coming soon here.

In the mean time, you can create sendgrid-php.php in your root directory and include the following:

<?php
require __DIR__ . '/vendor/autoload.php';
?>

With Best Regards,

Elmer

@nzhome
Copy link
Author

nzhome commented Jun 23, 2017

yeah I already did that an hour ago. I'm not a complete idiot. thanks.

@nzhome
Copy link
Author

nzhome commented Jun 23, 2017

and why do you insist on a script generating such a simple, yet crucial, file?

@thinkingserious
Copy link
Contributor

thinkingserious commented Jun 23, 2017

Hi @nzhome,

I apologize if you viewed my help as offensive, that was not my intent.

The person who created that script left the company a long time ago, so I'm not entirely sure why. My guess is that it was excluded since it's not necessary for those who are using composer. I think the fix in the PR will resolve this issue moving forward.

Thanks for taking the time to bring it to our attention.

With Best Regards,

Elmer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants