search

ignited / laravel-omnipay

Omnipay for Laravel & Lumen
MIT License
509 stars 100 forks source link

Laravel 8 #59

Closed mattcookdev closed 4 years ago

mattcookdev commented 4 years ago

Other than allowing v8 for illuminate/support, I also removed php-http/guzzle6-adapter since Guzzle 7 which is required for Laravel 8 supports the php-http implementation directly so no need for an adapter (php-http/guzzle6-adapter also causes a collision with Guzzle 7). Didn't seem like anything else needed changing.

I'm not sure what the consequences of this are for older versions of Laravel like 5 and 6, so hopefully someone else can look over this before merging. :)

mattcookdev commented 4 years ago

@alexw23 Is this all good to merge? Rely on this for a project and it's the only thing left preventing me from upgrading to laravel 8.