Skip to content

Commit

Permalink
Merge pull request #39 from grnhse/GREEN-36842_upgrade_guzzle
Browse files Browse the repository at this point in the history
Upgrade Guzzle version; Allow for v2/3 endpoints.
  • Loading branch information
tdphillipsjr authored Oct 11, 2022
2 parents dac20e0 + a81ed1f commit a7c6045
Show file tree
Hide file tree
Showing 6 changed files with 592 additions and 560 deletions.
6 changes: 3 additions & 3 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ jobs:
- image: circleci/php:<< parameters.php-version >>-node-browsers
steps:
- checkout
- run: sudo apt update
- run: sudo docker-php-ext-install zip
- setup_remote_docker:
version: 20.10.17
- restore_cache:
keys:
- v1-dependencies-{{ checksum "composer.json" }}
Expand All @@ -31,4 +31,4 @@ workflows:
- test:
matrix:
parameters:
php-version: ["7.3", "7.4"]
php-version: ["7.3", "7.4", "8.0"]
4 changes: 4 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,10 @@ This is available on Packagist. Install via Composer. Add the following to you
"grnhse/greenhouse-tools-php": "~2.0"
```

# New In 2.2
- Represents a minor version upgrade from Guzzle 7.1 to 7.5.
- Adds support for versions of Greenhouse endpoints beyond v1.

# New In 2.1
In order to support Greenhouse's new Inclusion URLs, we dropped a requirement that an $id be included for some URLs. Previously, a URL like `getActivityFeedForUser()` would throw a Greenhouse Exception with a message that id was required. Now, that method will just translate to `user/activity_feed` and return a 404 Not Found response. We did this in order to support Greenhouse's new Inclusion URLs. In the previous version `getQuestionsSetsForDemographics()`, [(found here)](https://developers.greenhouse.io/harvest.html#get-list-demographic-question-sets), would have thrown an "id required" exception instead of correctly routing to `demographics/question_sets`.

Expand Down
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
],
"require": {
"php": ">=7.3",
"guzzlehttp/guzzle": "~7"
"guzzlehttp/guzzle": ">=7.4.5"
},
"require-dev": {
"phpunit/phpunit": "*",
Expand Down
Loading

0 comments on commit a7c6045

Please sign in to comment.