php-kit icon indicating copy to clipboard operation
php-kit copied to clipboard

Upgrade to Guzzle 7.x

Open robindrost opened this issue 5 years ago • 6 comments

Hello,

Is it possible to support Guzzle 7.x? It was released in June. Currently new Laravel applications are set to Guzzle 7.x in the composer.json wich results in not being able to install this package.

The current work around is to downgrade Guzzle, but I rather see it gets supported here :)!

robindrost avatar Nov 06 '20 07:11 robindrost

Is there any update on when this is coming?

markwalet avatar Jun 07 '21 12:06 markwalet

Would be nice to get some feedback from Prismic...

goppi0 avatar Jul 14 '21 20:07 goppi0

I've personally given up with this library and written this - it's opinionated, but I actively maintain it. It uses PSR standards for everything, so you can use whatever http client you like.

gsteel avatar Jul 14 '21 21:07 gsteel

Finally someone who just takes over.

It's a shame that a first party package for a paid service is this poorly maintained... thanks for offering an alternative @gsteel

markwalet avatar Jul 14 '21 22:07 markwalet

Thanks @gsteel - hopefully Prismic makes use of the helping hand - after all, albeit they don't seem to reply on Github, the same issue was raised on the forum and someone from Prismic responded that they won't maintain the php sdk anymore and want to hand it over to the community to maintain. That was a while ago though and no further attempt was made by them to help their customers. Another package I found is zumba/prismic-sdk. Haven't compared it with yours, but by the sounds of it, it's not a complete rewrite, but just patched version. Again, Prismic should support people that want to help out....

goppi0 avatar Jul 15 '21 10:07 goppi0

Cheers @goppi0 - the api client I've written is a complete rewrite - I contributed a lot to this library but it just got less type safe and more out of date - I'm using my home-grown client in a lot of production projects and it's pretty stable.

I saw the "handing it over to the community" response but no one has announced anything or reached out to anyone that I'm aware of. It's all about the JavaScript now 🙄

gsteel avatar Jul 15 '21 10:07 gsteel

Why isn't the PR from @c0nst4ntin merged yet? It adds support for Guzzle 7.

ahm750 avatar Dec 28 '22 17:12 ahm750

I can merge the PR myself. I have all the rights needed. I was just waiting to hear back from @gsteel (probably once the Holliday's have passed) to be sure everything ist alright.

Hopefully this will be released shortly 😉

c0nst4ntin avatar Dec 28 '22 17:12 c0nst4ntin

Awesome, thanks!

ahm750 avatar Dec 28 '22 17:12 ahm750

Hey @c0nst4ntin, hate to be a bugger but wanted to check when the PR would be merged.

We upgraded our Laravel website recently and the only thing that's broken is our blog which is powered by Prismic.

Thanks!

ahm750 avatar Jan 04 '23 06:01 ahm750

Sorry for the long wait! But we now officially support Guzzle 7. If you run into any trouble please let us know immediately or open up a new Issue!

c0nst4ntin avatar Jan 06 '23 16:01 c0nst4ntin