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

The following changes relate to use in a non-composer environment: #633

Merged
merged 1 commit into from
Aug 7, 2018

Conversation

apcro
Copy link
Contributor

@apcro apcro commented Jun 27, 2018

Fixes

Checklist

  • I have read the [Contribution Guide] and my PR follows them.
  • I updated my branch with the master branch.
  • I have added necessary documentation about the functionality in the appropriate .md file

Short description of what this PR does:

The following changes relate to use in a non-composer environment:

  • Add missing file extension for /mail/BatchId in loader.php
  • Re-order requires to ensure EmailAddress is loaded before Bcc (Bcc depends on EmailAddress being loaded)
  • Change README.md to reference ./lib/loader.php instead of ./sendgrid-php.php for non-composer use.
  • Change example documentation to reference the required HTTP library dependency and how to install it without composer

NOTE: It is likely that the use commands in https://github.com/sendgrid/sendgrid-php/blob/master/USE_CASES.md#transactional-templates can be removed as well.

* Add missing file extension for `/mail/BatchId` in `loader.php`
* Re-order requires to ensure `EmailAddress` is loaded before `Bcc` (`Bcc` depends on `EmailAddress` being loaded)
* Change `README.md` to reference `./lib/loader.php` instead of `./sendgrid-php.php` for non-composer use.
* Change example documentation to reference the required HTTP library dependency and how to install it without composer
@thinkingserious thinkingserious added the status: code review request requesting a community code review or review from Twilio label Jun 27, 2018
@thinkingserious
Copy link
Contributor

Ah, it looks like you got it all sorted out after all! :)

I've added this to our backlog for a code review. Thanks!

@thinkingserious
Copy link
Contributor

Hello @apcro,

Thanks again for the PR!

We want to show our appreciation by sending you some swag. Could you please fill out this form so we can send it to you? Thanks!

Team SendGrid DX

@thinkingserious thinkingserious mentioned this pull request Aug 7, 2018
6 tasks
@apcro apcro deleted the non-composer-fixes branch September 22, 2018 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: code review request requesting a community code review or review from Twilio type: bug bug in the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants