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

Use Guzzle 4.x #19

Open
and1truong opened this issue Jul 1, 2014 · 12 comments
Open

Use Guzzle 4.x #19

and1truong opened this issue Jul 1, 2014 · 12 comments

Comments

@and1truong
Copy link

Can this library updated to Guzzle 4.x? Thanks.

@bakura10
Copy link
Member

bakura10 commented Jul 1, 2014

Hi,

Is there any reason to do so? I didn't try Guzzle 4 yet, do you think the change would be easy to do?

@and1truong
Copy link
Author

Is there any reason to do so?

For new projects, people will select Guzzle 4.x (quite stable for now). But it will conflict with this library.

do you think the change would be easy to do?

I did not check API changed yet. But I think it would not be hard.

@bakura10
Copy link
Member

bakura10 commented Jul 1, 2014

AWS is planning to do so for its SDK. I'm going to evaluate this today. I have severe libraries I need to update. I may try on this one.

It won't conflict because Guzzle 3 and Guzzle 4 are in different namespaces, so you can use both projects interchangeably. However, the simple fact that Guzzle 4 is 30% faster makes it a valid reason to update everything.

@and1truong
Copy link
Author

👍

@bakura10
Copy link
Member

bakura10 commented Jul 1, 2014

Hi again,

I'm facing several problems currently with Guzzle 4, especially related to exceptions :). I've raised a few issues to the Guzzle 4 developer. Hopefully ZfrMailChimp will be released later tonight or tomorrow with Guzzle 4 support!

@bakura10
Copy link
Member

bakura10 commented Jul 2, 2014

Hi again @andytruong ,

I've reverted the work on Guzzle 4 for now. This bug is blocking to me: guzzle/command#14

Furthermore the fact that Commands and Services part of Guzzle 4 are still in early development according to Jeremy leads to the fact that I'll likely wait a bit more (maybe one or two months) before migrating modules to Guzzle 4. I prefer to wait a bit so that everything stabilizes :).

@indeyets
Copy link

indeyets commented Jan 6, 2015

guzzle is 5.x already. and it is awesome!

@bakura10
Copy link
Member

bakura10 commented Jan 6, 2015

I've started migrating ZfrStripe to Guzzle 5 (zf-fr/zfr-stripe#14), but there are still a few things I'm having a hard time to replicate (handling exceptions error).

@bakura10
Copy link
Member

Hi everyone,

I don't have the time right now to do the migration for ZfrMailChimp (it's pretty big!). However, I'm nearly done with ZfrStripe, and if someone want to take some hours to follow what I did for ZfrMailChimp, this would be HIGHLY appreciated.

There are a few things to do:

If someone want to tackle this, please let me know!

@trsteel88
Copy link

+1

@bakura10
Copy link
Member

As guzzle 6 is now here.... I'll also try to actually upgrade to the v3 API of mailchimp :).

@bakura10
Copy link
Member

I do not have time to upgrade that unfortunately, but Guzzle Services has been tagged to 1.0.0 so upgrading to Guzzle 6 is now possible.

For those who'd be interested in upgrading this package, I've already upgraded my package ZfrShopify (https://github.com/zf-fr/zfr-shopify) to this new version. It should contain some hint on how to do it.

However there are still some issues like this one (guzzle/guzzle-services#129) that make me think we should wait a bit :).

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

4 participants