search

zendesk / zendesk_api_client_php

Official Zendesk API v2 client library for PHP
336 stars 259 forks source link

Allow guzzle 7 #453

Closed gmponos closed 4 years ago

gmponos commented 4 years ago

Hi,

We have recently tagged a new version 7 of guzzle.

https://github.com/guzzle/guzzle/releases/tag/7.0.0

🎉

aglipanci commented 4 years ago

@miogalang can you please merge this?

ipalaus commented 4 years ago

Can we get this merged? We can't update our app to Laravel 8 because we're using this package https://packagist.org/packages/huddledigital/zendesk-laravel and it's failing because Guzzle requirements on the underlying API (this repository). It's just a wrapper around and Laravel 8 now requires Guzzle 7.0.

aglipanci commented 4 years ago

I spoke with Zendesk Support and it seems like this package is no longer being maintained.

ipalaus commented 4 years ago

Thanks for the heads up!

derekps commented 4 years ago

CC: @miogalang

Adding more support to this. This is a very simple change that will allow compatibility with Laravel 8

ahmednasir91 commented 4 years ago

Indeed its a small change and required by many customers should be merged to unblock upgrade to laravel 8.0 @miogalang

derekps commented 4 years ago

Tagging top contributors

@miogalang @joseconsador @atroche @devarispbrown

gkatechis commented 4 years ago

Hi all! I just wanted to let you know that we will be merging this soon, we're just working on figuring out exactly how to handle some things internally, since this repo is technically no longer maintained by us. Thanks for your patience and we'll provide an update soon!

gkatechis commented 4 years ago

This update has been merged and I've bumped the latest version to 2.2.10

Thanks again all for your patience with this!