-
Notifications
You must be signed in to change notification settings - Fork 259
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
Allow guzzlehttp/psr7 2.0 #480
Conversation
With zendesk#469 being merged, this package now supports 1.7+ and 2.0
We are unable to use the zendesk client without this. Any update on merging it? |
+1 to merge this PR as |
Another +1 |
+1 |
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 ... 🙏 |
+1 |
Dear Zendesk, please merge this. |
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. |
+1 |
Hello Guys, I just fork the project see https://github.com/PululuK/zendesk_api_client_php Thanks |
Thank you @PululuK but I expect from the company to not ignore their official client. |
@willshen8 @yoshdog @manted what can I do to help you merge this PR? |
Not sure what needs to happen to get this thing merged. This PR exists since January, I've even opened an issue #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. |
This fix is necessary to run the Laravel 8, which depends on Monolog and requires PS7 ^2.0. |
@7ail @willshen8 @yoshdog any help? |
With #469 being merged, this package now supports 1.7+ and 2.0