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

Update Google API Library #10

Open
wants to merge 56 commits into
base: master
Choose a base branch
from
Open

Update Google API Library #10

wants to merge 56 commits into from

Conversation

roborourke
Copy link

Switching to composer based installation of the google PHP API Client.

@roborourke
Copy link
Author

@joehoyle you may want to take a look here too. All works well enough although the fetch gave me an out of memory error so may need to check on that.

I've committed the vendor folder, I know it's not ideal but means the plugin will work out of the box on clone for anyone not using a build process.

@mattheu
Copy link
Member

mattheu commented Jan 7, 2016

Hey looks good - this project got a little abandoned - but I'd done a bunch of work to update the google stuff over on the refactor branch: https://github.com/humanmade/hm-top-posts/tree/refactor

Also done some work to optimize this, as it didn't really scale well at all and I was running into lots of problems. Didn't realize anyone was using it!

@roborourke
Copy link
Author

@mattheu cheers for looking :) I'm extending it further still with some multisite related features eg. top blogs and top posts for the whole network.

This branch is built on top of the refactor branch so all the work you've done so far is in there too. Going to finish up in the next day or so so we can get it tested & ready before the 20th cutoff

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