Skip to content
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

can't use guzzlehttp 6.x,please update dependecy #97

Closed
argb opened this issue Oct 30, 2015 · 18 comments
Closed

can't use guzzlehttp 6.x,please update dependecy #97

argb opened this issue Oct 30, 2015 · 18 comments

Comments

@argb
Copy link

argb commented Oct 30, 2015

I can't this sdk,because you still use old guzzlehttp 5.x,it's really sorry for that.

@colin-stannard
Copy link

This is a problem for me too. Any chance of an upgrade soon?

@argb
Copy link
Author

argb commented Nov 8, 2015

@colin-stannard Finally ,I removed all packages depend on Guzzlehttp 6.

@RubenKruiswijk
Copy link

You could also use "mailgun/mailgun-php": "1.7.2" (which relies on the older guzzle/http) and guzzlehttp 6 together while waiting on the rewrite (see pull requests)

@argb
Copy link
Author

argb commented Nov 9, 2015

@RubenKruiswijk Thank you for your advices.

@DonnieDrummer
Copy link

I'm too need support guzzlehttp ~6.0

@perajovic
Copy link

+1

@joshfoskett
Copy link

Would love to see support for Guzzle 6, this is holding me back.

@brynnb
Copy link

brynnb commented Jan 11, 2016

Also looking forward to this, having to remove mailgun from my project until guzzle is updated/replaced. Thanks for all the hard work!

@RubenKruiswijk
Copy link

@DonnieDrummer, @perajovic, @RastaLulz, @brynnb please see my comment above. Version 1.7.2 is working just fine and provides the same functionality.

@brynnb
Copy link

brynnb commented Jan 11, 2016

@RubenKruiswijk thanks for the note, unfortunately I am relying on another library that relies on 1.8+, I am not using mailgun-php directly, so the lack of guzzle 6 is still an issue for me.

@olsgreen
Copy link

olsgreen commented Feb 5, 2016

+1 using @RubenKruiswijk work around atm.

@uaoleg
Copy link

uaoleg commented Feb 26, 2016

+1

v1.7.2 has some weird constants implementation which causes such an error:

PHP Notice 'yii\base\ErrorException' with message 'Use of undefined constant RECIPIENT_COUNT_LIMIT - assumed 'RECIPIENT_COUNT_LIMIT''

in \vendor\mailgun\mailgun-php\src\Mailgun\Messages\MessageBuilder.php:88

@Nyholm
Copy link
Collaborator

Nyholm commented Mar 5, 2016

PR #94 will allow you to decide what version of Guzzle to use. Please review it and test it so it will be easy for the maintainers to make a decision to merge or not.

@mollierobbert
Copy link

+1 it should work with guzzlehttp 6.

@DSpeichert
Copy link

+1

2 similar comments
@renzell
Copy link

renzell commented Mar 31, 2016

+1

@dbebawy
Copy link

dbebawy commented Mar 31, 2016

+1

@Nyholm
Copy link
Collaborator

Nyholm commented Jul 20, 2016

With Guzzle 6 (and 5) can be used in the version 2.0

@Nyholm Nyholm closed this as completed Jul 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests