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

Guzzle 5 #49

Merged
merged 1 commit into from
Nov 27, 2014
Merged

Guzzle 5 #49

merged 1 commit into from
Nov 27, 2014

Conversation

GrahamCampbell
Copy link
Contributor

Why not? :P

Sorry, can't regenerate the lock file - doing this from the web interface.

@manavo
Copy link
Contributor

manavo commented Nov 27, 2014

Will merge and update the lock file!

manavo added a commit that referenced this pull request Nov 27, 2014
@manavo manavo merged commit 8ccad62 into cachethq:master Nov 27, 2014
@manavo
Copy link
Contributor

manavo commented Nov 27, 2014

Lock file updated. Thanks @GrahamCampbell :)

@jbrooksuk
Copy link
Member

I wasn't sure if there was anything I'd need to upgrade in the code itself, glad it's this easy.

@manavo
Copy link
Contributor

manavo commented Nov 27, 2014

Only used at the moment in the Webhook model, but that's already broken anyway.

Tries to access $this->client, which is undefined.

Edit: ignore me, I forgot I fixed that yesterday...

@jbrooksuk
Copy link
Member

Yeah I was supposed to create an instance of the client in the constructor.

@GrahamCampbell GrahamCampbell deleted the patch-1 branch November 27, 2014 20:01
@flemzord flemzord mentioned this pull request May 20, 2015
@GrahamCampbell GrahamCampbell modified the milestone: V0.1.0 Alpha Jul 25, 2015
@efriel efriel mentioned this pull request May 30, 2017
@raffus raffus mentioned this pull request Mar 22, 2018
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.

3 participants