Guzzle upgrade to v6 - Githubissues

jelovac / bitly4laravel

Provides a Laravel package to communicate with Bit.ly API

35 stars 18 forks source link

Guzzle upgrade to v6 #23

Closed jelovac closed 7 years ago

jelovac commented 9 years ago

Greetings,

This issue concerns the upgrade of Guzzle library to v6 used by this package.

The upgrade plan is the following:

change the composer file to require v6
migrate guzzle v5 logic to v6

rgds, Vladimir

trevorgehman commented 8 years ago

Any update on this?

mbarwick83 commented 8 years ago

+1 for an update on this

jelovac commented 8 years ago

Sorry guys for the late update, I currently don't have spare time to develop this myself so if you create a good pull request I would gladly approve it.

Kind regards, Vladimir

mbarwick83 commented 8 years ago

@MyoStudio has a fixGuzzleVersionTo6.0 branch that I'm using. Works like a charm. Need to json_decode() response though - as Guzzle 6 doesn't have the ->toJson() or whatever it was < 6. https://github.com/MyoStudio/bitly4laravel