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

Add google login #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add google login #11

wants to merge 1 commit into from

Conversation

ishmael
Copy link

@ishmael ishmael commented Feb 8, 2013

Hi,

Yesterday had to add this to my project and decided to try and give something back.

Hope it's not too bad :)

Thanks

@spectrumbranch
Copy link

+1

4 similar comments
@HarasimowiczKamil
Copy link

+1

@julianjelfs
Copy link

+1

@twickstrom
Copy link

+1

@designerkamal
Copy link

+1

@ben-ng
Copy link

ben-ng commented Sep 18, 2013

Someone yell at me if i haven't merged this in by Sunday (:

@SilverPaladin
Copy link

Yell @ben-ng!

@ben-ng
Copy link

ben-ng commented Nov 19, 2013

thanks for the yell @SilverPaladin (:

it looks like this will be more difficult to merge than anticipated. really unfortunate that we didn't take care of this earlier before the sources diverged too much.

i've pushed a partial merge to the googleauth branch. if someone wants to take a look at app/helpers/actions.js and fix the conflicts there, we'll be good to go.

@dlabey
Copy link
Contributor

dlabey commented Jul 12, 2015

@ben-ng can you update your repo?

@dlabey
Copy link
Contributor

dlabey commented Jul 13, 2015

Hi guys actually OpenID is now deprecated and it seems there is Google OAuth support now so this is invalid.

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.

None yet

9 participants