search

ovh / php-ovh-sms

PHP for ovh sms API
Other
48 stars 49 forks source link

Update OVH API to 2.0.x #20

Closed dsferruzza closed 5 years ago

dsferruzza commented 7 years ago

Hi!

This PR upgrades the dependency on the ovh/ovh package to its last version. This allows to have Guzzle 6.x and solves #13.

It tested the package before and after this commit (using the example code in the README and real credentials). I got similar results in both cases so it seems to work!

I hope you will merge it and tag a new version (even a pre-release would help!). Let me know if I can do something more.

benjamineukles commented 7 years ago

+1, I definitively need it. Can anyone tell me when this will be merged into the master?

kblais commented 7 years ago

@benjamineukles hope it will be merged soon... Needing this for a Laravel project with Guzzle 6...

benjamineukles commented 7 years ago

@kblais as noticed by @dsferruzza you can use his fork here #13

kblais commented 7 years ago

@benjamineukles Yeah, that's what we're gonna do while the PR is not merged, but using the "official" version up to date is still the best :)

nWidart commented 7 years ago

Any news on this?

AClavijo commented 6 years ago

Any news on this?

Maxeee09 commented 6 years ago

+1 Need it !