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

Change client-implementation to client-common. #200

Closed
wants to merge 1 commit into from

Conversation

noud
Copy link

@noud noud commented Sep 17, 2018

Change php-http/client-implementation to php-http/client-common
because that package changed it's name.

This will solve issue:
composer missing dep: php-http/client-implementation ^1 #197

And composer is again able to install omnipay-common again.

Would you please be so kind to accept this pull request?

Thanks,
Noud

Change php-http/client-implementation to php-http/client-common
because this package changed it's name.
@clash82
Copy link

clash82 commented Sep 21, 2018

Can someone review and merge this, pls? I got the same issue.

@judgej
Copy link
Member

judgej commented Sep 21, 2018

@barryvdh I think you may be the guy for this one. As it stands, it all works for me, so I really don't understand the problem. Not saying there isn't a problem - just don't understand it.

Is this a production (non-dev) only issue?

@barryvdh
Copy link
Member

You're just missing a client implementation. This is a virtual package. See the readme

@barryvdh barryvdh closed this Sep 21, 2018
@noud
Copy link
Author

noud commented Sep 21, 2018

@barryvdh
Copy link
Member

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

Successfully merging this pull request may close these issues.

4 participants