zendesk / zendesk_api_client_php

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

Allow guzzlehttp/psr7 2.0 #480

Closed wouterj closed 2 years ago

wouterj commented 2 years ago

With #469 being merged, this package now supports 1.7+ and 2.0

ianwelsh commented 2 years ago

We are unable to use the zendesk client without this. Any update on merging it?

deluxetom commented 2 years ago

+1 to merge this PR as guzzlehttp/psr 2.0 is supported

gjuric commented 2 years ago

Another +1

PululuK commented 2 years ago

+1

baiiko commented 2 years ago

Hey @7ail

You can merge this request and up a new version ? :)

BR,

Edit: A teammate fork this repo with this fix, and it's work please ... :pray:

cuongthetran commented 2 years ago

+1

ethanbeyer commented 2 years ago

Dear Zendesk, please merge this.

oojacoboo commented 2 years ago

It's well over 3 months since this very simple PR has been submitted. There is no excuse for this to not have already been merged.

sbacelic commented 2 years ago

+1

PululuK commented 2 years ago

Hello Guys,

I just fork the project see https://github.com/PululuK/zendesk_api_client_php I can add new maintainers if you are motivated to contribute. I will release soon :)

Thanks

PululuK commented 2 years ago

See :

gjuric commented 2 years ago

Thank you @PululuK but I expect from the company to not ignore their official client.

wouterj commented 2 years ago

@willshen8 @yoshdog @manted what can I do to help you merge this PR? At the moment, this PR is creating noise in both your and my GitHub inbox. At the same time, it is blocking all of Zendesk's users to upgrade their stack.

gjuric commented 2 years ago

Not sure what needs to happen to get this thing merged. This PR exists since January, I've even opened an issue https://github.com/zendesk/zendesk_api_client_php/issues/487 almost two months ago and there was no communication from the company.

In the meantime I've contacted Zendesk support since we are paying for the platform after all but they were unresponsive.

Right now, I've just checked Zendesk documentation https://developer.zendesk.com/documentation/ticketing/api-clients/php/ and it looks like this client is not officially supported by Zendesk, it would be nice if they stated that in the Readme.

For those of you tired of waiting I propose that you either check @PululuK 's fork or check Packagist docs for Loading a package from a VCS repository.

jasonlav commented 2 years ago

This fix is necessary to run the Laravel 8, which depends on Monolog and requires PS7 ^2.0.

simonmaass commented 2 years ago

@7ail @willshen8 @yoshdog any help?

PululuK commented 2 years ago

image